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

Update outdated doc link #2799

Merged
4 commits merged into from
Mar 10, 2021
Merged

Update outdated doc link #2799

4 commits merged into from
Mar 10, 2021

Conversation

vritant24
Copy link
Member

Addresses this devcom feedback

The previous link pointed to an outdated doc link where the "Recommended Version" led to a dead end.
I have updated the link to point to the same doc in a new location

@Haplois
Copy link
Contributor

Haplois commented Mar 6, 2021

It might be easier to update where https://go.microsoft.com/fwlink/?LinkID=236877&clcid=0x409 is pointing. Let me first try that.

Copy link
Contributor

@Haplois Haplois left a comment

Choose a reason for hiding this comment

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

Trying to update the aka.ms link.

Copy link
Contributor

@Haplois Haplois left a comment

Choose a reason for hiding this comment

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

Instead of updating the old fwlink, just created a new one and used that. https://aka.ms/tp/vstest/multitargetingdoc?view=vs-2019

@Haplois Haplois self-assigned this Mar 10, 2021
@ghost
Copy link

ghost commented Mar 10, 2021

Hello @Haplois!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 35d2279 into master Mar 10, 2021
@ghost ghost deleted the dev/vrbhardw/updateMultiTargetLink branch March 10, 2021 10:51
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants