Skip to content
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

Remove partial users after verifying otp #73

Open
aditya-navana opened this issue May 25, 2021 · 0 comments
Open

Remove partial users after verifying otp #73

aditya-navana opened this issue May 25, 2021 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@aditya-navana
Copy link
Collaborator

We are saving the access code of the users in the DB as soon as it is verified. This is useful for the user flow but causes an issue where an user which is partially registered (verified the access code but didn't complete the OTP submission) can be saved which causes the app to go to the multi-user screen when there is only one fully registered user.

This is quite unrealistic in a real world use case since all the users would only receive one access code. However, we can solve this problem by removing all the partial users as soon the user otp verification is complete. Feel free to reach out to me if you need more clarity on the issue.

@aditya-navana aditya-navana added bug Something isn't working good first issue Good for newcomers labels May 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant