-
Notifications
You must be signed in to change notification settings - Fork 289
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
fix(clerk-js): Improve UX when adding adding phone MFA #4860
fix(clerk-js): Improve UX when adding adding phone MFA #4860
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: 81d2ffd The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
36baf18
to
c9edb75
Compare
738efe9
to
81d2ffd
Compare
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.
💯
81d2ffd
to
e598d63
Compare
Description
This PR improves the UX when adding a new phone as an MFA strategy, after the changes when verifying a phone number it will automatically be added as an MFA option
Before
CleanShot.2025-01-09.at.19.59.42.mp4
After
CleanShot.2025-01-09.at.19.55.16.mp4
Checklist
pnpm test
runs as expected.pnpm build
runs as expected.Type of change