Skip to content

test: update data connector tests to support new linking#4431

Open
lorenzo-cavazzi wants to merge 3 commits intorelease-2.17.0from
lorenzo/data-connector-modal
Open

test: update data connector tests to support new linking#4431
lorenzo-cavazzi wants to merge 3 commits intorelease-2.17.0from
lorenzo/data-connector-modal

Conversation

@lorenzo-cavazzi
Copy link
Copy Markdown
Member

@lorenzo-cavazzi lorenzo-cavazzi commented Apr 15, 2026

Update data connectors tests to support the new search modal when linking/adding data connectors

/deploy

@lorenzo-cavazzi lorenzo-cavazzi requested a review from a team as a code owner April 15, 2026 11:20
@RenkuBot
Copy link
Copy Markdown
Collaborator

You can access the deployment of this PR at https://ci-renku-4431.dev.renku.ch

Base automatically changed from release-2.16.0 to master April 15, 2026 14:17
@olevski olevski requested review from a team as code owners April 15, 2026 14:17
@olevski
Copy link
Copy Markdown
Member

olevski commented Apr 15, 2026

@lorenzo-cavazzi release 2.16.0 was merged please move to #4434

@lorenzo-cavazzi lorenzo-cavazzi changed the base branch from master to release-2.17.0 April 21, 2026 08:39
@lorenzo-cavazzi lorenzo-cavazzi changed the base branch from release-2.17.0 to master April 21, 2026 12:01
@lorenzo-cavazzi lorenzo-cavazzi force-pushed the lorenzo/data-connector-modal branch from c7db2b2 to 9f2075b Compare April 21, 2026 12:05
@lorenzo-cavazzi
Copy link
Copy Markdown
Member Author

@lorenzo-cavazzi release 2.16.0 was merged please move to #4434

Ok 👍
I'll keep master as the target until the 2.17.0 PR is updated, then I'll swap and we can merge this (the UI has been tagged to this PR is fully ready)

visibility: "private",
}).then((response) => {
projectId = response.body.id;
projectId = response.body.id ?? "";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why? If the response is not typed, maybe add types.

Copy link
Copy Markdown
Member Author

@lorenzo-cavazzi lorenzo-cavazzi Apr 29, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that Project is defined with id?: string;. Perhaps it's used for patches?

I would rather not change the function used in all other tests and limit this PR's scope to the UI feature. I pushed this f6c2ecb 61437eb and we can change that in a separate PR if we wish

@lorenzo-cavazzi lorenzo-cavazzi force-pushed the lorenzo/data-connector-modal branch from f6c2ecb to 61437eb Compare April 29, 2026 14:56
@lorenzo-cavazzi lorenzo-cavazzi changed the base branch from master to release-2.17.0 April 29, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants