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

Move dashboard to /dashboard #229

Open
2 of 3 tasks
dahlbyk opened this issue Mar 10, 2016 · 3 comments
Open
2 of 3 tasks

Move dashboard to /dashboard #229

dahlbyk opened this issue Mar 10, 2016 · 3 comments

Comments

@dahlbyk
Copy link
Member

dahlbyk commented Mar 10, 2016

I like Segment's approach - landing on the home page, even if logged in, shows their marketing site. Clicking Login immediately takes you to your workspaces.

I propose that we:

  • Add /dashboard route for the dashboard.
  • Remove the logic to show the dashboard on / if you're authenticated.
  • Make /login redirect to /dashboard if you're already authenticated.
@drusellers
Copy link
Member

Github.com does something similar too don't they?

@dahlbyk
Copy link
Member Author

dahlbyk commented Mar 10, 2016

GitHub.com does what we do today, hijacking / for the current user's activity stream. It's one of the few apps to do so, that I can think of off-hand.

One of my primary motivations here is to differentiate between use of the Marketing site and use of the app. Instead of redirecting, we could instead hijack the page tracking to pretend that we're at /dashboard.

But more generally, if someone is logged in there is no way for them to see the Marketing site (e.g. if an OSS-only user wanted to check Pricing to use HB at their company).

@dahlbyk dahlbyk added this to the Engineering milestone Apr 15, 2016
This was referenced May 4, 2016
@discorick
Copy link
Member

Moving to 'next release' as this is almost complete already

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

No branches or pull requests

4 participants