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

Documentation about persisting prerendered state is deceptive #31275

Closed
viliml opened this issue Dec 20, 2023 · 3 comments · Fixed by #31276
Closed

Documentation about persisting prerendered state is deceptive #31275

viliml opened this issue Dec 20, 2023 · 3 comments · Fixed by #31276
Assignees
Labels
8.0 .NET 8 Blazor doc-enhancement Pri1 Source - Docs.ms Docs Customer feedback via GitHub Issue

Comments

@viliml
Copy link

viliml commented Dec 20, 2023

Description

The component PrerenderedCounter2.razor as described in the guide does not work in the case of enhanced navigation (dotnet/aspnetcore#51584), which is most cases since enhanced navigation is on by default.
Developers who try to follow the guide to implement persisting prerendered state will be confused why it doesn't work.

Page URL

https://learn.microsoft.com/en-us/aspnet/core/blazor/components/prerender?view=aspnetcore-8.0

Content source URL

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/components/prerender.md

Document ID

ffdc8df3-55a8-b757-d4e6-38c104916002

Article author

@guardrex

Copy link
Contributor

🍂🎁 Happy Holidays! ❄️⛄

A green dinosaur 🦖 will be along shortly to assist. Stand-by ........

@guardrex
Copy link
Collaborator

Thanks, @viliml ... I'll address this as soon as I can, but it might be after the holiday 🎆 break.

In the meantime to surface this issue and the PU issue, I'll drop a quick note into the article right now.

@guardrex guardrex moved this from Triage to 8.0 in Blazor.Docs Dec 20, 2023
@guardrex
Copy link
Collaborator

Actually, I think I can FIX this right now. Javier's remarks can be provided and the PU issue cross-linked for more information.

@guardrex guardrex moved this from 8.0 to In progress in Blazor.Docs Dec 20, 2023
@github-project-automation github-project-automation bot moved this from In progress to Done in Blazor.Docs Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.0 .NET 8 Blazor doc-enhancement Pri1 Source - Docs.ms Docs Customer feedback via GitHub Issue
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants