Skip to content

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Oct 6, 2025

Fixes #36181

Thanks @Markz878! 🚀

I'm fleshing out your brief example with a fully working, cut-'n-paste experience.

Two paragraphs are moving up because I'm introducing some new subheadings to make the distinctions among scenarios better.


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/state-management/prerendered-state-persistence.md aspnetcore/blazor/state-management/prerendered-state-persistence
aspnetcore/blazor/state-management/server.md aspnetcore/blazor/state-management/server

@guardrex guardrex self-assigned this Oct 6, 2025
@guardrex guardrex requested a review from Copilot October 6, 2025 13:50
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the state persistence service documentation by providing a complete, working example that developers can copy and paste. It addresses issue #36181 by expanding the previously brief service example into a comprehensive demonstration with full code snippets.

Key changes:

  • Adds a complete working example for the CounterTracker service with detailed implementation
  • Reorganizes content structure with new subheadings for better clarity between different persistence scenarios
  • Moves explanatory paragraphs to a more logical position in the document flow

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
aspnetcore/blazor/state-management/server.md Adds TODO comment for future API documentation updates
aspnetcore/blazor/state-management/prerendered-state-persistence.md Expands service persistence example with complete implementation, adds new section headers, and reorganizes content for better flow

@guardrex guardrex requested review from javiercn, tdykstra and wadepickett and removed request for javiercn October 6, 2025 13:56
Copy link
Contributor

@wadepickett wadepickett left a comment

Choose a reason for hiding this comment

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

@guardrex, Looks great! I can't find any issues. Well done.

@guardrex
Copy link
Collaborator Author

guardrex commented Oct 9, 2025

Thanks Wade! 🎸

@guardrex guardrex merged commit 4477d29 into main Oct 9, 2025
3 checks passed
@guardrex guardrex deleted the guardrex/blazor-prerendering-service-example branch October 9, 2025 19:03
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.

Provide a valid RegisterPersistentService example
2 participants