-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Update links (remove en-us+update MSDN links) #2580
Comments
Make sure you keep links inside the microservices book absolute. |
@mairaw "microservices book" == all of the topics under Are there any other areas of the TOC where we want absolute links? |
Is this and #2037 duplicates? |
@BillWagner These updates started with merely the goal of dropping I think #2037 covers dealing more thoroughly with the entire MSDN and fwlink situation, and we've only done that lightly here. The scripts I've used grab the canonical link from redirected responses to drop MSDN links where possible (especially the GUID-based ones in favor of the library-based ones), but as @mairaw already corrected on one of the PR's here, there's apparently many of them that won't get their correct OPS links. They simply didn't redirect to an OPS topic, so the script took the MSDN page at its library-based endpoint address for the link. I think #2037 should be left open and worked on after this issue is resolved. The work on this issue pushes things in the right direction imo. |
@mairaw Unassigning now ... I won't be able to address this issue further in the time remaining with the issues that I can finish by Monday night. |
Closing this since we have #2037 tracking this change more broadly. |
@mairaw I'll break these into batches on separate PRs:
azure.microsoft.com
ones, too! Update Azure links #2635 (merged)The text was updated successfully, but these errors were encountered: