-
-
Notifications
You must be signed in to change notification settings - Fork 189
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 username #752
Add username #752
Conversation
Hi @Palakkgoyal can you please resolve conflicts? If you face issues, let me know. |
Sure. |
aeed2bd
to
20f9a12
Compare
I solved the merge issue. Can you please check. |
1 similar comment
I solved the merge issue. Can you please check. |
new pr? or should I reopen this?? |
I didn't closed this. It get closed automatically. Should I make a new one? |
nono I reopened it, so it's fine now, will review and merge in a while |
Ohk. Thanks a lot. |
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.
looks good to me
Thanks a lot for fixing this! |
No worries. |
This PR closes issue no. #574
Done all the required changes such as checking whether the user exist or not. If not then, create a new user doc in users collection and if does exists then, check whether they have a username or not. If not then, give them uid as their username and if exists then, do not do anything in both sign in and sign up page.