-
Notifications
You must be signed in to change notification settings - Fork 28
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
Implement MSC2965 action parameter #1673
Conversation
frontend/src/Router.tsx
Outdated
|
||
const [route, setRoute] = useAtom(routeAtom); | ||
if (redirect) { | ||
setRoute(redirect.route, redirect.searchParams); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be in a useEffect, because you're not supposed to do state changes in the render?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kerry won't see this before Monday, sadly, and we have to land this PR today. @t3chguy could you help us out by giving advice?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can take care of finishing that up, alongside the server side of this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes I believe it should be in a useEffect
Deploying with
|
Latest commit: |
651d948
|
Status: | ✅ Deploy successful! |
Preview URL: | https://9e670dbd.matrix-authentication-service-docs.pages.dev |
Branch Preview URL: | https://kerry-1550-logout-deep-link.matrix-authentication-service-docs.pages.dev |
Alright, I:
|
ec61b6e
to
651d948
Compare
Ported from MAS, thank you @sandhose See matrix-org/matrix-authentication-service#1673
Fixes #1550
Redirects to session detail when query param
action=session_end
is encountered.Screen.Recording.2023-09-01.at.14.48.35.mov