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

Make hyperlinks work #4821

Merged
merged 1 commit into from
Nov 26, 2021
Merged

Conversation

clyang82
Copy link
Contributor

@clyang82 clyang82 commented Nov 1, 2021

Signed-off-by: clyang82 chuyang@redhat.com

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

Verification

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

This is by design, like this and it's done by our automation. IF some links does not work, it should be fixed within automation project. What is not working? From GH or from website?

cc @saswatamcode

@saswatamcode
Copy link
Member

saswatamcode commented Nov 1, 2021

I think these links (plus any non-markdown resource links) are broken on the website but fine on GH. An automatic solution for this was already implemented in mdox but the config wasn't updated here.

It just involves adding the line below to .mdox.yaml,

linkPrefixForNonMarkdownResources: "https://github.com/thanos-io/thanos/blob/main"

Maybe it can be updated in this PR? Or I can raise new one? 🙂
@bwplotka @clyang82

@clyang82
Copy link
Contributor Author

clyang82 commented Nov 2, 2021

Thanks @bwplotka @saswatamcode for your comments. Good to know we have automatic solution for this. Let me use this PR to update .mdox.yaml.

Signed-off-by: clyang82 <chuyang@redhat.com>
@clyang82 clyang82 force-pushed the correct_objectstore_link branch from 334033b to f293bf0 Compare November 2, 2021 06:02
@clyang82
Copy link
Contributor Author

clyang82 commented Nov 2, 2021

/assign @saswatamcode

Copy link
Member

@saswatamcode saswatamcode left a comment

Choose a reason for hiding this comment

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

Thank you! All the links that were broken seem to be fixed now! 💫

Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@yeya24 yeya24 merged commit 78391de into thanos-io:main Nov 26, 2021
@clyang82 clyang82 deleted the correct_objectstore_link branch November 29, 2021 02:40
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.

4 participants