Skip to content

Conversation

@amcasey
Copy link
Member

@amcasey amcasey commented Aug 31, 2023

  1. We shouldn't be squatting on a general URL like http://aka.ms/dataprotectionwarning - instead, use https://aka.ms/aspnet/dataprotectionwarning
  2. Some messages are already clear and don't require URLs
  3. The motivating example in Add aka.ms links to data protection warnings #22029 hadn't been updated to include the URL

Part of #22029.

@ghost ghost added area-dataprotection Includes: DataProtection labels Aug 31, 2023
@RichardD2
Copy link

What's the correct URL here?

For me, http://aka.ms/dataprotectionwarning redirects to https://github.com/dotnet/aspnetcore, and https://aka.ms/aspnet/dataprotectionwarning redirects to https://www.bing.com/?ref=aka&shorturl=aspnet/dataprotectionwarning, which is just an empty Bing search page. Neither link provides the promised "more information".

@amcasey
Copy link
Member Author

amcasey commented Sep 5, 2023

@RichardD2 If we decide to go ahead with this PR (which is still a draft), we'll create an aka.ms link for https://aka.ms/aspnet/dataprotectionwarning and point it at relevant documentation.

@amcasey amcasey marked this pull request as ready for review September 6, 2023 18:16
@adityamandaleeka adityamandaleeka merged commit 716bfef into dotnet:release/8.0 Sep 6, 2023
@ghost ghost added this to the 8.0-rc2 milestone Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-dataprotection Includes: DataProtection

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants