Feature/waa s prompt user to sign in with previous login method #44
Conversation
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
…n. Created a scene for WaaS end to end testing. Made LoginPanel a prefab to make it easier to re-use accross scenes (as I use it in the WaaSEndToEndTests scene
… Add a basic transfer and signing test
…ound in the scene and the LoginPanel is dismissed
…ceptions thrown throughout and fail the test - otherwise, the tests fail silently and the WaaSTestHarness never outputs the results (as it doesn't receive all the expected pass/fail events
…and manually for email sign in) when loging in to WaaS. Store this in PlayerPrefs as an enum/int. Load it in the LoginPage awake method so that we can behave accordingly
… a user is most likely to login via that method
…via social login or email login). If a user most recently logged in via email but goes to log in with a different email, open the InfoPopupPanel and notify the user that they are logging in with a different email than last time. Similarly, if the user last logged in with email, automatically fill the email field with the email the user used last. For social login, include the email most recently used as the login_hint during the request.
taylanpince
approved these changes
Jan 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Save the method and email user used to log in on the most recent session using PlayerPrefs. Highlight the most recent method so that user is most likely to log in the same way. If the user doesn't login via that method, display a popup to remind the user how they logged in last time and notify them that if they use a different login method than they will be given a different account.
Scenario 1 - User uses same login method as last time:
https://github.com/0xsequence/sequence-unity/assets/49766546/aefe0d1e-037c-4c50-b990-71849ed5ea93
Scenario 2 - User previously logged in via social sign in and goes to log in via email (or a different social sign in provider)
https://github.com/0xsequence/sequence-unity/assets/49766546/6653d800-5d26-43f7-9b33-673ad900487c
Scenario 3 - User previously logged in via email and goes to log in via social sign in (or a different email)
https://github.com/0xsequence/sequence-unity/assets/49766546/18fbd44d-8144-44a1-b61f-862ced5d18c8