-
Notifications
You must be signed in to change notification settings - Fork 100
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
Google sign in with some warning changes #265
Conversation
@akankshagoel28 is attempting to deploy a commit to the ommaniya70's projects Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for your Pull Request! We truly appreciate your contribution! 🙌 🌟 Thank you for your efforts in improving the project! 🙏 |
@akankshagoel28 Why you have modified other pages also? You just have to work on your assigned task only. Raise new PR and modify only assigned page (Login page). Note: Work on latest repository |
there were a lot of warnings when i ran the repo that is why, i changed the name since i thought it would be easier to understand |
@akankshagoel28 why you modified these files? Ignore the warnings and modify just login and signup page. You can modify signup page only if it is required. also, Do you also want to work on Forgot Password issue? |
alright i will work on those issues and modify only the login page with the forgot password feature |
Firstly I have added the google button on the sign in page, i also made some changes in the naming to get a clear idea and made some changes for I was getting a lot of warnings on my terminal, there are still a lot of warnings, if you want i will create a new issue for that