-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: change the help page url #1667
Conversation
src/www/ui_components/app-root.js
Outdated
@@ -372,7 +372,7 @@ export class AppRoot extends mixinBehaviors([AppLocalizeBehavior], PolymerElemen | |||
[[localize('about-page-title')]] | |||
</paper-item> | |||
<paper-item name="help"> | |||
<a href="https://s3.amazonaws.com/outline-vpn/index.html#/support" id="helpAnchor" hidden=""></a> | |||
<a href="https://getoutline.org/how-it-works" id="helpAnchor" hidden=""></a> |
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.
Let's switch it to https://support.getoutline.org instead, since that's a better destination for a user who's looking for help.
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.
Changed the link to the new one, sorry for long reply.
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.
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.
We don't have an S3 mirror of the Support site, and AFAIK it hasn't gotten blocked, even when getoutline.org has (but correct me if I'm wrong).
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.
See comment on new link destination
6c12f79
to
34fbda5
Compare
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.
LGTM
Codecov ReportPatch has no changes to coverable lines.
📢 Thoughts on this report? Let us know!. |
Thanks for the update, @andreevgy . Please sync the branch with the latest master branch before merge. Or we can do this for you if you prefer. |
34fbda5
to
6751ace
Compare
@jyyi1 just rebased it, should be good to go. |
Hello again!
Noticed that help link was not working for a while. Changed it to the working page on current website. Let me know if there should be something different.
P.S.: There is some issue with intl config on getoutline.org because it redirects me to
/intl/[lang]
and show me this error