Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport of [ui] When your token expires and you sign in again, redirect to your original route. into release/1.9.x #24390

Conversation

hc-github-team-nomad-core
Copy link
Contributor

Backport

This PR is auto-generated from #24374 to be assessed for backporting due to the inclusion of the label backport/1.9.x.

The below text is copied from the body of the original PR.


When your token 403s or 500s (when it's deleted behind the scenes, or expires either while you're using the app, or while it's in your localStorage, say, overnight), this PR adds functionality to remember the page (and queryParams) you were on when it happened.

Upon signing back in (with a renewed token, say), you'll be automatically rerouted back to that page, with a little notification saying so.

Resolves #23793


Overview of commits

@hc-github-team-nomad-core hc-github-team-nomad-core force-pushed the backport/23793-ui-when-a-token-is-expired-upon-re-authentication-take-me-back-to-the-route-i-came-from/incredibly-big-marlin branch from efcb4c2 to fabe218 Compare November 7, 2024 15:44
Copy link

github-actions bot commented Nov 7, 2024

Ember Test Audit comparison

release/1.9.x fabe218 change
passes 1577 1580 +3
failures 5 5 0
flaky 0 0 0
duration 000ms 000ms -000ms

@philrenaud philrenaud merged commit ee55848 into release/1.9.x Nov 7, 2024
16 of 17 checks passed
@philrenaud philrenaud deleted the backport/23793-ui-when-a-token-is-expired-upon-re-authentication-take-me-back-to-the-route-i-came-from/incredibly-big-marlin branch November 7, 2024 20:28
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.

2 participants