-
Notifications
You must be signed in to change notification settings - Fork 45
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 signup with email #93
Conversation
Github is showing the branch has some conflicts. but if i remove the code it will give error. what should i do ? @Mohitmadhav |
Did you do git pull upstream main before pushing your changes? |
|
yes this is the only conflict showing to me. |
|
I did. But i got this -> Please make sure you have the correct access rights So I did git pull origin main and I got ->
|
try out Then again git pull upstream main |
@keyurgit45 I'm getting null uid when I login |
I implemented sign up with Email not login. Also the app is missing log out feature. |
Yeah, I assigned Google Sign out to someone else. |
Sure, I would Like to work on them. |
Cool, the issue will be ready within a few minutes. |
Thank you for your help. This is my first open-source contribution. Also, can you remove my email id's from firebase that I used for testing purpose? |
No problem bro. |
Description
Summary of the changes and which issue is fixed.
I have added sign-up with email and password (firebase). The code for the same is in lib\screens\auth. Added validation for email and password. Also modified the Custom TextEditingField to use textEditingController in order to use the email and password for further sign-up use.
Fixes #83
Type of change
Checklist: