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

Dead link #4878

Merged
merged 2 commits into from
Apr 5, 2018
Merged

Dead link #4878

merged 2 commits into from
Apr 5, 2018

Conversation

user135711
Copy link
Contributor

Closest possible link I found is Programming with Application Domains and Assemblies but that's the head topic of the side bar which seems unnecessary to link to. Edit the link as desired.

Closest possible link I found is [Programming with Application Domains and Assemblies](https://docs.microsoft.com/en-us/dotnet/framework/app-domains/) but that's the head topic of the side bar which seems unnecessary to link to. Edit the link as desired.
@pkulikov
Copy link
Contributor

pkulikov commented Apr 4, 2018

The link to the older version (.NET Framework 4) works:
https://msdn.microsoft.com/en-us/library/bd36055b-56bd-43eb-b4d8-820c37172131(v=vs.100)

Should that one replace the broken one?
Or that one from the docs is better:
https://docs.microsoft.com/en-us/dotnet/framework/app-domains/use

@user135711 please notice that the same dead link appears in several See Also sections in how-to topics

@mairaw
Copy link
Contributor

mairaw commented Apr 4, 2018

Actually that topic seemed to be rolled into this one now:
https://docs.microsoft.com/en-us/dotnet/framework/app-domains/application-domains#programming-with-application-domains

So you can link like this: "application-domains.md#programming-with-application-domains"

@mairaw mairaw closed this Apr 5, 2018
@mairaw mairaw reopened this Apr 5, 2018
There are five more of this link as noted. I can't fix them because I'm aborting a failed merge.
Copy link
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.

Thank you @user135711. I'll fix the remaining ones. Your fix should go live within a day.

@mairaw mairaw merged commit 0ac2547 into dotnet:master Apr 5, 2018
@mairaw
Copy link
Contributor

mairaw commented Apr 5, 2018

Related to #2037

@user135711 user135711 deleted the patch-6 branch April 5, 2018 21:42
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.

3 participants