Skip to content

feat: add mfa flow to the expo app #1544

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

blakecduncan
Copy link
Collaborator

@blakecduncan blakecduncan commented Apr 17, 2025

Pull Request Checklist


PR-Codex overview

This PR introduces Multi-Factor Authentication (MFA) functionality to the application, enhancing security by allowing users to verify their identity through an authenticator app. It includes new screens and updates to existing components to handle MFA processes.

Detailed summary

  • Added a new MFAScreen component for managing MFA settings.
  • Updated OTPAuthScreen to include MFA verification.
  • Introduced handleMfaSubmit function for MFA code validation.
  • Updated Tabs in _layout.tsx to include an MFA tab.
  • Modified OAuth logic to support MFA.
  • Updated dependencies in package.json to newer versions.
  • Enhanced SplashScreen.storyboard with multiple background color entries.
  • Adjusted styles for better UI presentation in MFA screens.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Apr 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aa-sdk-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2025 5:22pm
aa-sdk-ui-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2025 5:22pm

@moldy530
Copy link
Collaborator

can you post a video of the flow

Base automatically changed from blake/rn-signer-mfa-methods to main April 21, 2025 22:19
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