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

GitHub Relative Links do not work on docs.microsoft.com/samples #6977

Closed
willmtemple opened this issue Jan 15, 2020 · 4 comments
Closed

GitHub Relative Links do not work on docs.microsoft.com/samples #6977

willmtemple opened this issue Jan 15, 2020 · 4 comments
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Docs
Milestone

Comments

@willmtemple
Copy link
Contributor

When our readme pages are generated from the markdown for ingestion into docs.microsoft.com, relative links are not made relative to the document's location on github (as they would be if they were displayed here on github). Instead, they convert into broken links that are relative to the page on docs.microsoft.com.

See this example : https://docs.microsoft.com/en-us/samples/azure/azure-sdk-for-js/keyvault-certificates-typescript/

... in which the purgeAllCertificates.ts link is broken.

Seems that we have to take extra care to make sure link URI's in material that is rendered through docs.microsoft.com/samples are absolute.

@willmtemple willmtemple self-assigned this Jan 15, 2020
@willmtemple willmtemple added Docs and removed triage labels Jan 15, 2020
@ramya-rao-a ramya-rao-a added the Client This issue points to a problem in the data-plane of the library. label Jan 15, 2020
@ramya-rao-a ramya-rao-a added this to the [2020] February milestone Jan 15, 2020
@xirzec xirzec modified the milestones: [2020] February, [2020] April Mar 9, 2020
@ramya-rao-a
Copy link
Contributor

@willmtemple Is the action item here that we need to have all the readmes reviewed?
@czubair Is this something you can add to the doc review checklist you have for each release?

@willmtemple
Copy link
Contributor Author

@ramya-rao-a the whole situation with links on README files is weird. We really probably want to use links that are tagged to a specific commit when anything is published to docs.microsoft.com, but I don't think that functionality exists.

For now, I've used a regular expression to find files that have relative links in the markdown files:

⇒  rg -lg "sdk/*/*/samples/*/README.md" "\]: \."
sdk/keyvault/keyvault-keys/samples/javascript/README.md
sdk/keyvault/keyvault-keys/samples/typescript/README.md
sdk/keyvault/keyvault-secrets/samples/typescript/README.md
sdk/keyvault/keyvault-secrets/samples/javascript/README.md
sdk/keyvault/keyvault-certificates/samples/javascript/README.md
sdk/keyvault/keyvault-certificates/samples/typescript/README.md

A bunch of the package READMEs also have relative links, but they are handled correctly by NPM and don't show up on docs.microsoft.com as far as I can tell.

@weshaggard
Copy link
Member

Related to Azure/azure-sdk#1214.

@ramya-rao-a
Copy link
Contributor

Thanks @weshaggard

@willmtemple, am closing this issue in favor of the upstream one

@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. Docs
Projects
None yet
Development

No branches or pull requests

4 participants