Navigation Menu

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

[Enhancement] You're all logged in! #196

Closed
Drewster727 opened this issue Dec 6, 2015 · 4 comments
Closed

[Enhancement] You're all logged in! #196

Drewster727 opened this issue Dec 6, 2015 · 4 comments
Assignees
Labels

Comments

@Drewster727
Copy link

This may seem trivial, but I feel like the "You're all logged in!" page should not exist at all. If a user is already logged in it should default them to the "Search" page automatically, IMO.

Thanks!

@lokenx
Copy link
Owner

lokenx commented Dec 6, 2015

I agree! It was added recently because by putting a redirect in the back button behavior would get messed up and you'd get stuck on a page. I intend to look into how better to route things in the new version to handle this.

@Foebik
Copy link

Foebik commented Jan 8, 2016

I know in the past you shorten the timer on this page, I think it could even go shorter. 1 second, 1.5 seconds?

@dannyhanes
Copy link

^^ This. If I am already logged in just take me to /search. Thanks!

@lokenx lokenx added the bug label Mar 17, 2016
@lokenx lokenx self-assigned this Mar 17, 2016
@DejaVu
Copy link

DejaVu commented Apr 5, 2016

Further to this, I've been told by a Windows Phone user that clicking TV Shows in Search sends him to a "You're all logged in page". He cannot request TV Shows because of it on his phone.

@lokenx lokenx closed this as completed in 05165b1 Sep 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants