-
Notifications
You must be signed in to change notification settings - Fork 9
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
No phone Auth flow #801
No phone Auth flow #801
Conversation
return .run { send in | ||
await send(.internal(.loadProfileOutcome(loadProfile()))) | ||
} | ||
return verifyPasscode() |
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.
Verify the passcode again when user taps unlock. This was the main source of the bug, as loading the profile will succeed and then the user will be redirected to home screen,
state.biometricsCheckFailed = true | ||
errorQueue.schedule(error) |
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.
Show any error, in some cases the kechain will throw the error that there is no keychain configured
.
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.
NICE! LGTM!
Tested locally on iOS 17 device (only where I wanted to remove passcode), worked great.
https://radixdlt.atlassian.net/browse/ABW-2335
handle no auth flows + moving gateways check to main + some dependencies clean up.
Demo:
RPReplay_Final1695826982.MP4