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

Article does not explain which file should be edited #34143

Closed
dougclutter opened this issue Nov 15, 2024 · 0 comments · Fixed by #34146
Closed

Article does not explain which file should be edited #34143

dougclutter opened this issue Nov 15, 2024 · 0 comments · Fixed by #34146
Assignees

Comments

@dougclutter
Copy link

dougclutter commented Nov 15, 2024

Description

In the section titled Blazor WebAssembly authentication uses history state for redirects, the first razor code snippet has a heading like this:

The following example demonstrates the prior redirection approach for apps that target .NET 6 or earlier, which is based on a redirect URL (?returnUrl=) with NavigateTo:

It does not explain that the file to be updated is RedirectToLogin.razor; it would be very helpful if the filename was included.

Fortunately, the third razor code snippet does include the file to be updated: Shared/LoginDisplay.razor.

Page URL

https://learn.microsoft.com/en-us/aspnet/core/migration/60-70?view=aspnetcore-7.0&tabs=visual-studio#blazor-webassembly-authentication-uses-history-state-for-redirects

Content source URL

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/migration/60-70.md

Document ID

e9669d35-3de1-997d-258c-745ebbdbface

Article author

@Rick-Anderson

Related Issues

@guardrex guardrex self-assigned this Nov 15, 2024
@github-project-automation github-project-automation bot moved this to Triage in Blazor.Docs Nov 15, 2024
@guardrex guardrex moved this from Triage to P0/P1 - High Priority in Blazor.Docs Nov 15, 2024
@guardrex guardrex moved this from P0/P1 - High Priority to In progress in Blazor.Docs Nov 16, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Blazor.Docs Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants