Skip to content

Add an email code as an MFA factor - #64

Open
pacific-ring wants to merge 2 commits into
mainfrom
feature/mfa-email-otp
Open

pacific-ring wants to merge 2 commits into
mainfrom
feature/mfa-email-otp

Conversation

@pacific-ring

Copy link
Copy Markdown
Contributor

Description

The second-factor list now offers an email address the server names, next to push and a phone code. The code screen is shared with the phone code, so a wrong code, Back, "Verify another way" and Resend behave the same.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvement

Testing

  • Unit tests
  • Manual testing
  • Sample app testing

Checklist

The second-factor list now offers an email address the server names, next
to push and a phone code. The code screen is shared with the phone code,
so a wrong code, Back, "Verify another way" and Resend behave the same.
Factors are listed as iOS lists them: push, text, voice call, then email.

Co-Authored-By: Claude <noreply@anthropic.com>
…ck while verifying

Co-Authored-By: Claude <noreply@anthropic.com>
Comment thread EXAMPLES.md
### Multi-Factor Authentication

When the tenant's MFA policy asks for more than the emailed code, `EmbeddedLoginScreen` moves into MFA in place; nothing extra is needed in your app. Today it runs a push notification or a code sent to a phone, by text message or voice call. With several factors, the user picks one from a list first. Other factors aren't supported in this flow yet.
When the tenant's MFA policy asks for more than the emailed code, `EmbeddedLoginScreen` moves into MFA in place; nothing extra is needed in your app. Today it runs a push notification, or a code sent to a phone (by text message or voice call) or to an email address. With several factors, the user picks one from a list first. Other factors aren't supported in this flow yet.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nit: Pbly just adding a list of supported factors will be easier to update

This branch has not been deployed

No deployments
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