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

Fix redirect bug to /# that occurs on some browsers #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mndewitt
Copy link

@mndewitt mndewitt commented May 7, 2015

Hi Amos,

Great work on the book, I learned a tremendous amount! I built out an app from the book and life was good. I checked back in about 6 weeks later on Chrome, and I came across the same bug described here: linnovate/mean#192, when trying to login with Google. It only happened on Chrome, and it became an issue only in the last 6 weeks as I had done all my development on Chrome. Anyways, I implemented the fix described in the issue successfully, and I think it would be really valuable to include it in your repo since you include a fix for the Facebook redirect anyways. This problem completely befuddled me for a couple of hours (everything was working in Firefox and Safari), and would be maddening for someone to debug when they are working through your book...

Thanks again for your work!

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.

1 participant