Skip to content

Converted msdn links to site-relative for offline scenarios#3896

Merged
mairaw merged 7 commits intodotnet:masterfrom
mikeblome:mb-msdn9
Dec 9, 2017
Merged

Converted msdn links to site-relative for offline scenarios#3896
mairaw merged 7 commits intodotnet:masterfrom
mikeblome:mb-msdn9

Conversation

@mikeblome
Copy link
Copy Markdown
Contributor

This PR fixes up msdn shortid urls that were missed in earlier runs

@mikeblome mikeblome changed the title converted msdn links to site-relative for offline scenarios [WIP] converted msdn links to site-relative for offline scenarios Dec 8, 2017
attempt at link fix
Copy link
Copy Markdown
Contributor Author

@mikeblome mikeblome left a comment

Choose a reason for hiding this comment

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

@mairaw @BillWagner Link report showed 93, two above baseline of 91, and I attempted to fix one of those inline, so should just be one additional that I introduced possibly. I request that we accept anyway as a net plus in terms of correct links.

@mairaw
Copy link
Copy Markdown
Contributor

mairaw commented Dec 8, 2017

Thanks so much for working on this @mikeblome. We'll review it as soon as possible!

Copy link
Copy Markdown
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

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

Thanks @mikeblome. I did a couple of fixes as well that might bring us back to 91. We'll wait for builds to finish before merging.

@mairaw
Copy link
Copy Markdown
Contributor

mairaw commented Dec 8, 2017

Related to #2037

@mairaw mairaw changed the title [WIP] converted msdn links to site-relative for offline scenarios Converted msdn links to site-relative for offline scenarios Dec 9, 2017
@mairaw mairaw merged commit f09f76f into dotnet:master Dec 9, 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.

2 participants