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

Fixed link #41

Merged
merged 2 commits into from Jun 6, 2017
Merged

Fixed link #41

merged 2 commits into from Jun 6, 2017

Conversation

philipp-classen
Copy link
Contributor

The landing page shows the link to the dashboard as
http://green-analytics.test.cliqz.com/dashboard/?token=...
which results in 404 Not found.

Changed it, so it shows http://green-analytics.test.cliqz.com/dashboard?token=...
(Did not change it, but https://... would be even better)

@@ -164,7 +164,7 @@ <h1 class="brand-heading">Green Analytics</h1>


<h4>Accessing Dashboard</h4>
<p> <a href='#'>{{ context.dashboard }}/?token={{ context.site_key }}</a> ,
<p> <a href='#'>{{ context.dashboard }}?token={{ context.site_key }}</a> ,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@philipp-classen : Can you update the protocol to https ? It should be fine now, given that we have https enabled. The best way would be the change the protocol in .env file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if there are side-effects when we set PROTOCOL=https in .env. If I'm not mistaken, it should work, though.

@konark-cliqz konark-cliqz merged commit b72ef62 into cliqz-oss:master Jun 6, 2017
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