-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Wrap-up 10.0 inline work items #36495
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
Conversation
There was a problem hiding this 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 completes the remaining 10.0 Blazor inline work items by removing TODO comments and adding proper API cross-references. The changes primarily involve converting placeholder text like "UPDATE 10.0 - MIA API" to actual xref syntax for API documentation, updating version tracking comments from 10.0 to 11.0, and completing documentation that was waiting on upstream PRs.
Key changes:
- Converted API placeholders to proper xref cross-references for navigation and component state APIs
- Updated version tracking in HTML comments from 10.0 to 11.0 for items deferred to next release
- Completed deferred documentation sections that were awaiting PR merges
- Enhanced form validation example with complete code samples
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
aspnetcore/blazor/state-management/prerendered-state-persistence.md |
Replaced placeholder comment with proper xref to RegisterOnRestoring API |
aspnetcore/blazor/security/blazor-web-app-with-oidc.md |
Updated version tracking comment from 10.0 to 11.0 for deferred issue |
aspnetcore/blazor/project-structure.md |
Removed hold comment and added proper xref for NotFound component documentation |
aspnetcore/blazor/progressive-web-app/index.md |
Removed obsolete workshop reference comment and added xref to push notifications article |
aspnetcore/blazor/fundamentals/routing.md |
Replaced API placeholders with proper xref syntax for Router and NavigationManager APIs |
aspnetcore/blazor/fundamentals/navigation.md |
Extensive API cross-reference updates for NavigationManager, NavLink, and Router components |
aspnetcore/blazor/forms/validation.md |
Added complete code examples for OrderItem and ShippingAddress classes for nested validation |
aspnetcore/blazor/components/quickgrid.md |
Updated version tracking comment from 10.0 to 11.0 for scaffolding issue |
aspnetcore/blazor/call-web-api.md |
Updated inline comment to track as 11.0 work item for missing API documentation |
Co-authored-by: Copilot <[email protected]>
wadepickett
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved. Looks good.
Fixes #34437
🎉
This wraps up most of the remaining 10.0 Blazor work.
Reminder: Article dates aren't updated in the Blazor node for minor changes, which include all of the changes on this PR.
There are two known items left, which are tracked separately: a single bit of API to surface in the API Browser and sample app work (on the samples repo) conducted by the PU. There's one item that I just need to check on that might not lead to updates, which pertains to some Not Found feature updates that Ilona and I made prior to release (will check on it after this merges).
Internal previews