Added Email Auth(#27)

[puruganda]

No description provided.

[welcome]

Hello there!👋 Welcome to the project!💖
Thank you and congrats🎉 for opening your first pull request. Please make sure you have gone through our Code of Conduct and contributing files before making this pr. 🙌🙌 We will get back to you as soon as we can 😄.

[himanshusharma89]

@puru-ganda Update the PR as per the PR template and do attach a video of the auth too.

[puruganda]

@puru-ganda Update the PR as per the PR template and do attach a video of the auth too.

@himanshusharma89 should i also need to implement sign up functionality??

[puruganda]

@himanshusharma89 can you please add a USER to the firebase console and provide its EMAIL and PASSWORD details to me so that I can use that for verification in demo video

[himanshusharma89]

@puru-ganda Email auth is enabled now.

[puruganda]

@puru-ganda Email auth is enabled now.

@himanshusharma89 okay, but I need a added USER email and password so that I can verify the email auth functionality

[himanshusharma89]

@himanshusharma89 okay, but I need a added USER email and password so that I can verify the email auth functionality

Yeah, I almost forgot it. I'll create an issue for that and then you can start implementing it on that screen

[puruganda]

@himanshusharma89 okay, but I need a added USER email and password so that I can verify the email auth functionality

Yeah, I almost forgot it. I'll create an issue for that and then you can start implementing it on that screen

okay

[himanshusharma89]

@puru-ganda Pull the latest code, the screen is added. You can start working on it

[puruganda]

@puru-ganda Pull the latest code, the screen is added. You can start working on it

sure

[Rishabh510]

@himanshusharma89 we can link this to an issue for it to get tracked

[puruganda]

Related Issue

• Fixes Firebase Authentication #27 .

Proposed Changes

• Added User Email Auth in login screen
• Added User Registration using email and password in sign up screen
• Added logout functionality
• Added cutom_dialogalert widget for Auth functionality

Checklist

• Tested
• No Conflicts
• Change In Code
• Change In Documentation

Demo

email_auth.mp4

[himanshusharma89]

We need a few changes here:

• Show the error below the fields not in a Dialog
• Don't use the same controller for both the text fields
• Squash the commits: https://github.com/wprig/wprig/wiki/How-to-squash-commits
• Follow the lining rules, especially in signup.dart

[himanshusharma89]

@puru-ganda Pull the latest code before working on it.

[puruganda]

@puru-ganda Pull the latest code before working on it.

okay

[himanshusharma89]

@puru-ganda 30 files got changed for this PR, I think you didn't rebase your code perfectly, try it again. And also implement the email auth inside the AuthService class.

[puruganda]

@puru-ganda 30 files got changed for this PR, I think you didn't rebase your code perfectly, try it again. And also implement the email auth inside the AuthService class.

Yes I was facing some issues with merge, I will implement email auth soon.

[puruganda]

@himanshusharma89 Can I create a new PR and close this one as I am facing issues with the current local branch EmailAuth.
because I need to create a new branch and implement email auth on that.

[himanshusharma89]

@himanshusharma89 Can I create a new PR and close this one as I am facing issues with the current local branch EmailAuth.
because I need to create a new branch and implement email auth on that.

I don't encourage it, but go ahead, if you are not able to perform the task.

[puruganda]

@himanshusharma89 Can I create a new PR and close this one as I am facing issues with the current local branch EmailAuth.
because I need to create a new branch and implement email auth on that.

I don't encourage it, but go ahead, if you are not able to perform the task.

okay