-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Blazor Preview 7 coverage #35919
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
Blazor Preview 7 coverage #35919
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 updates Blazor documentation to cover features introduced in .NET 10 Preview 7. The changes primarily focus on updating API names, enhancing Not Found page handling, and documenting new features like Hot Reload for WebAssembly and the InputHidden component.
Key Changes
- Updated attribute names from
[SupplyParameterFromPersistentComponentState]
to[PersistentState]
across all documentation - Enhanced Not Found page handling with improved Router capabilities and removal of legacy NotFound render fragment support
- Added documentation for new features including Hot Reload for WebAssembly, InputHidden component, and PWA service worker improvements
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.
Show a summary per file
File | Description |
---|---|
aspnetcore/release-notes/aspnetcore-10/includes/blazor.md | Comprehensive updates covering all Preview 7 features including API renames, Router improvements, and new components |
aspnetcore/client-side/dotnet-interop.md | Added Hot Reload configuration guidance for WebAssembly apps |
aspnetcore/blazor/state-management/server.md | Updated persistent state attributes and circuit pause/resume API names |
aspnetcore/blazor/state-management/prerendered-state-persistence.md | Updated attribute names and added serialization extensibility documentation |
aspnetcore/blazor/security/index.md | Updated persistent state attribute usage in security context |
aspnetcore/blazor/progressive-web-app/index.md | Enhanced service worker registration with caching prevention |
aspnetcore/blazor/host-and-deploy/server/index.md | Updated component name from LinkPreload to ResourcePreloader |
aspnetcore/blazor/fundamentals/routing.md | Major updates to Not Found handling with new Router capabilities |
aspnetcore/blazor/forms/input-components.md | Added documentation for new InputHidden component |
aspnetcore/blazor/components/render-modes.md | Updated authorization handling for static SSR |
aspnetcore/blazor/components/lifecycle.md | Updated persistent state attribute usage |
aspnetcore/blazor/components/layouts.md | Simplified layout examples by removing deprecated NotFound references |
aspnetcore/blazor/components/integration.md | Updated persistent state attribute usage in integration scenarios |
Co-authored-by: Copilot <[email protected]>
Thx @maraf ... In addition to the changes on the last commit, could you also take a look at Line 236 of the PWA article to see if that original lead-in text is correct?
|
@danroth27 ... My plan is to have the passkeys coverage on a PR tomorrow by EOD 🤞, but I need your direction ... Also, I assume that you'll want this PR merged when Pre7 drops even if I don't get all of the reviews. Is that correct? |
@ilonatommy ... Are you able to take a look? ... and also see if you like those Copilot code suggestions ☝️. This is for everyone. I'm still waiting on any PU signoff 😆 ....................... — Bueller? ... Bueller? .............. Bueller? - Economics Teacher (Ben Stein) |
@ilonatommy ... I'm going to go ahead and accept those Copilot suggestions. As for the rest, @danroth27, I have no PU signoffs ... and we're out of time. I guess you want me to merge this for the release, but I don't have a ton of confidence in what's here without PU feedback. |
Co-authored-by: Copilot <[email protected]>
@wadepickett @tdykstra ... Pre7 dropped! 🎉🕺💃🪩 ... I'll merge this now. |
@wadepickett ... There are several NIT 😈 suggestions on the build report that must have gone in on today's earlier PRs ... They aren't Blazor related (i.e., related to this PR), so I'll go ahead here. |
Oh! ... and @wadepickett or @tdykstra ... I need a signoff from one of you just to merge this. |
Thanks @tdykstra! 🎷 That's a wrap for Blazor bits for today. I'll have the passkeys coverage up by EOD tomorrow. |
Fixes #35918
Fixes #35795
Fixes #35861
Fixes #35197
Fixes #35745
Fixes #35920
Fixes #35921
Fixes #35922
NOTES
It doesn't seem like this one requires doc updates (implementation detail) ...
Internal previews
Toggle expand/collapse