Skip to content

dbeaver/cloudbeaver#4417 add redirect on the same tab - #4502

Open
HocKu7 wants to merge 5 commits into
develfrom
dbeaver/cloudbeaver#4417-add-redirect-on-the-same-tab
Open

dbeaver/cloudbeaver#4417 add redirect on the same tab#4502
HocKu7 wants to merge 5 commits into
develfrom
dbeaver/cloudbeaver#4417-add-redirect-on-the-same-tab

Conversation

@HocKu7

@HocKu7 HocKu7 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Closes #4417

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 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.

sergeyteleshev
sergeyteleshev previously approved these changes Jul 29, 2026

if (redirectLinks.length) {
const url = redirectLinks[0];
const redirectLink = redirectLinks[0] as UserLogoutInfo['redirectLinks'][0];

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

pls add gql fragment for redirectLinks and reuse it here

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

actually type was not needed here. redirectLinks already typed right, so that was a hack to sooth the ts compiler. replaced with exclamation mark

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.

Reverse Proxy logout opens a popup window instead of redirecting in the same tab

5 participants