Skip to content

dbeaver/pro#9638 [CB] refresh is needed to see replaced connectionfolder in another instance#4464

Merged
devnaumov merged 11 commits into
develfrom
9638-cb-refresh-is-needed-to-see-replaced-connectionfolder-in-another-instance
Jul 23, 2026
Merged

dbeaver/pro#9638 [CB] refresh is needed to see replaced connectionfolder in another instance#4464
devnaumov merged 11 commits into
develfrom
9638-cb-refresh-is-needed-to-see-replaced-connectionfolder-in-another-instance

Conversation

@SychevAndrey

Copy link
Copy Markdown
Contributor

There are two parts. The first commit is just fixing stale reference and the second one is an improvement that we might want to have or not (need to discuss)

…Ds for active projects

Changed all four spots from reference equality to id comparison, matching the rest of the codebase, see #4430
@SychevAndrey SychevAndrey self-assigned this Jul 15, 2026
@codacy-production

codacy-production Bot commented Jul 15, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Comment thread webapp/packages/plugin-connections/src/NavNodes/ConnectionNavNodeService.ts Outdated
SychevAndrey and others added 3 commits July 15, 2026 14:35
Server events don't carry the actual data, so besides outdating (which lets the handlers invalidate the previous state, e.g. the old location in the navigation tree) we explicitly load the connection to guarantee that the handlers receive the actual state
Comment thread webapp/packages/core-connections/src/ConnectionInfoResource.ts
@devnaumov
devnaumov merged commit 9e68261 into devel Jul 23, 2026
6 checks passed
@devnaumov
devnaumov deleted the 9638-cb-refresh-is-needed-to-see-replaced-connectionfolder-in-another-instance branch July 23, 2026 15:33
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