-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Add conditional passwordless flow #13655
Merged
Merged
Changes from 25 commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
d7a5369
TODOs in signin page
Beamanator 1bdbe88
A few magic code translations
Beamanator 19033be
Passwordless betas
Beamanator 8822080
Initial new validate code form & action
Beamanator 145c457
Pass betas into SignInPage
Beamanator bb5b663
Cleanup to use passwordless permission check
Beamanator 25d4b9d
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator 0208f86
New magic code / 2fa translations
Beamanator 581f951
New validate code validation function
Beamanator aa6108d
Rename send validate code form
Beamanator 8579cfd
Update to use validateCodes & new UI
Beamanator 40f9fc0
Update sign in page with new form and text
Beamanator 0ea78ed
Remove unused error
Beamanator 52355cf
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator 13ff2b7
Remove sendValidateCode since it doesn't exist yet
Beamanator 119db2c
Add validation back, use resendValidateCode
Beamanator af94044
Get login to pass to ResendValidateCode
Beamanator 9e474ce
Make welcomeText logic more clear
Beamanator 3cccdd4
Pass password or validateCode to SigninUser
Beamanator 14eb2e3
Lint fixes
Beamanator 2ddee7b
Adapt signin, not sure what to do with accountID yet
Beamanator 35d729c
Add missing spanish translations
Beamanator 96017a1
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator ce91caa
Change ResendValidationForm name back
Beamanator 100b72b
Condense comment
Beamanator d777db3
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator f6420ed
Using informal lang (es)
Beamanator ba95d92
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator af761cd
Revert changes to this page
Beamanator d22df59
Resolve lint error
Beamanator 4600cc4
Remove comment causing lint issues
Beamanator f8c964a
Send isPasswordless param to resend validate code flow
Beamanator aba7bef
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator 3347bf8
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator 14eb86e
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator 8054dc3
Merge branch 'main' of github.com:Expensify/App into beaman-addCondit…
Beamanator c66879b
A few PR review comments
Beamanator b67c710
Update 2fa error message
Beamanator File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Also from the figma mockups (here) this should just be
Please enter your two factor code
since we're trying to limit the number of "code" references we're usingThere 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.
We use
recovery code
if they enter the incorrect 2FA code with an error message like so (on oldDot):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.
Huh I wonder if Figma was updated recently ish? I the original text here used to be accurate 😅 Hope I'm not crazy 🤞
And now it looks like the text should actually be
Please enter your authenticator code
to matchThere 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.
Requested Spanish translation here