-
Notifications
You must be signed in to change notification settings - Fork 340
fix(authenticator): redirect to sign-in after account confirmation #6789
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
base: main
Are you sure you want to change the base?
Conversation
|
Description of changes
Fix authenticator signup confirmation flow to redirect users to sign-in screen instead of signup screen when coming from refresh scenario. This improve the UX where users who refresh the page between signup and email confirmation are incorrectly redirected back to the signup flow after confirming their email.
Changes:
• Add
shouldManualSignInguard to detect refresh scenario (DONE + CONFIRM_SIGN_UP context)• Add
manualSignInstate that redirects to sign-in screen with setSignInStep action• Update
confirmSignUponDone handlers to include shouldManualSignIn condition• Add console logs for debugging and verification
Behavior:
• Normal flow: User signs up → confirms email → auto signed in ✅
• Refresh flow: User signs up → refresh → sign in → confirms email → redirected to sign-in screen ✅
Issue #, if available
Fixes inconsistent user experience in signup confirmation flow after page refresh.
Description of how you validated changes
• Tested both normal signup flow (auto sign-in works)
• Tested refresh scenario (now redirects to sign-in instead of signup)
Checklist
• [x] Have read the Pull Request Guidelines
• [x] PR description included
• [ ] yarn test passes and tests are updated/added
• [x] PR title and commit messages follow conventional commit syntax
• [x] If this change should result in a version bump, changeset added (This can be done
after creating the PR.) This does not apply to changes made to docs, e2e, examples, or other private packages.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.