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

adjusting login.html to match new google sign-in migration #84

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

LorenzoTomaz
Copy link

While i was building one of my udacity full stack nanodegree's project i realize that the code for login.html present in ud330/Lesson3/step3/templates/ was not working as suppose to, so after reading the new documentation for the google+ sign-in i've found a better way to construct login.html. The changes for the google+ sign-in authentication system can be viewed in https://developers.google.com/identity/sign-in/web/quick-migration-guide

@AbdelhamidGamal
Copy link

udacity doesn't even merge it's own employee pull requests to fix it's none working code , fantastic. thanks udacity for making learning such a horrible experience

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants