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
Combine marketing and product homepage #156
Conversation
New marketing homepageThis branch combines the marketing homepage – currently on about.sourcegraph.com – and the product homepage on sourcegraph.com/search. This solves a problem of users not knowing how to get to our product from our marketing page. This page is only visible on sourcegraph.com and when a no users are signed-in out. A user that is signed-in will get the normal product homepage. |
@@ -212,15 +212,10 @@ func serveHome(w http.ResponseWriter, r *http.Request) error { | |||
|
|||
if (r.Host == "sourcegraph.com" || r.Host == "www.sourcegraph.com") && !actor.FromContext(r.Context()).IsAuthenticated() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vanesa for having the ability to test, change this line to become:
if envvar.SourcegraphDotComMode() && !actor.FromContext(r.Context()).IsAuthenticated() {
Codecov Report
|
New marketing homepage for sourcegraph.com! Router users visiting about.sourcegraph.com/ and sourcegraph.com/ to the new homepage based off the following conditions: If a user is logged out -> / -> /start and gets the new homepage If a user is logged in -> / -> /search and gets the normal search page Users that visit either /start or /search will get the page without redirections This is not implemented on non-sourcegraph.com instances.
* Combine marketing and product homepage (#156) New marketing homepage for sourcegraph.com! Router users visiting about.sourcegraph.com/ and sourcegraph.com/ to the new homepage based off the following conditions: If a user is logged out -> / -> /start and gets the new homepage If a user is logged in -> / -> /search and gets the normal search page Users that visit either /start or /search will get the page without redirections This is not implemented on non-sourcegraph.com instances. * Invert check of sourcegraph.com for 404
.layout { | ||
display: block !important; | ||
} | ||
* { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like a severe change that could break many things?
from sourcegraph/sourcegraph-intermediate-2018@0df1ccd
fixes https://github.com/sourcegraph/enterprise/issues/13480
cc @francisschmaltz this is your new branch :)