-
Notifications
You must be signed in to change notification settings - Fork 125
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
feat: oauth add passkey after signup (popup flow) #1151
Open
linnall
wants to merge
46
commits into
main
Choose a base branch
from
linna/oauth-add-passkey-popup
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+26
−14
Open
Changes from 42 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
3882dca
chore: local dev changes updated
linnall 8ab130a
Merge branch 'main' into social-auth-dev
linnall ca9e5fd
chore: merge in main
linnall 0a63891
chore: wip oauth redirect with add passkey after signup
linnall f62b9c8
Merge branch 'main' into social-auth-dev
linnall 8b42698
Merge branch 'main' into social-auth-dev
linnall bf02b3c
chore: wip
linnall 5d5951a
refactor: remove a few things so we can test redirect
moldy530 ee46248
refactor: just render the pop-up on completed login for oauth
moldy530 13b1156
refactor: add new user signup event in signer
moldy530 1101154
refactor: add hook to listen to signup events
moldy530 c3aa3bf
chore: wip
linnall a559ab7
refactor: use signup event to popup passkey create
moldy530 f4a9d94
fix: the modal was staying open
moldy530 9b504a3
fix: move setAuthStep in the correct card
moldy530 0b592c0
fix: infinite subscribe loop
moldy530 d2c862c
fix: don't open passkey create until logged in
moldy530 979442b
Merge branch 'main' into social-auth-dev
linnall 998463e
chore: remove local dev callback url
linnall 1e04872
Merge branch 'main' into linna/oauth-add-passkey
linnall 85edb03
chore: removes unecessary bangs and refactors for readability
linnall a853832
chore: removes unused usePrevious hook
linnall 195149e
Merge branch 'main' into linna/oauth-add-passkey
linnall f061538
Merge branch 'main' into linna/oauth-add-passkey
linnall 6060220
fix: update useNewUserSignup to remove listener on dismount and when …
linnall c0e796a
docs: link related discussion to TODO
linnall e1dda2f
refactor: adds emitNewUserEvent wrapper
linnall 8d10d70
chore: restore oauth mode to popup
linnall 46f694e
Merge branch 'main' into linna/oauth-add-passkey
linnall 78e37ed
chore: remove logging
linnall 1975bdd
chore: remove logging
linnall d952810
chore: reset oauth mode to popup
linnall 640a7de
fix: handle undefined isNewUser within emitNewUserEvent wrapper
linnall 1adcbd7
chore: remove unnecessary bang
linnall 49309d8
Merge branch 'main' into linna/oauth-add-passkey
linnall 6b17b45
Merge branch 'main' into linna/oauth-add-passkey-popup
linnall c1f44c6
Merge branch 'main' into linna/oauth-add-passkey-popup
linnall d72fa1a
feat: sets up event for client to emit to trigger signer to emit new …
linnall ac8dc20
Merge branch 'main' into linna/oauth-add-passkey-popup
linnall f7f650b
Merge branch 'main' into linna/oauth-add-passkey-popup
linnall 995a210
chore: adds logging
linnall aa12a59
fix: solve the race condition on new user signup
moldy530 529fe0a
refactor: changes newUserSignupClient to newUserSignup
linnall 0364404
Merge branch 'main' into linna/oauth-add-passkey-popup
linnall e7a76d3
chore: remove console trace
linnall e3467ba
chore: remove empty line
linnall 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
Submodule embedded-accounts-quickstart
updated
4 files
+2 −0 | .gitignore | |
+7 −1 | next.config.mjs | |
+2 −2 | src/config.ts | |
+10 −0 | turbo.json |
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.
this solves a race condition where an on success handler would set the step to complete and close the modal that was just opened. This makes sure that all handlers complete we're done with the setup flow prior to add passkey