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

Cascading values/params and render mode boundaries 8.0 #30958

Merged
merged 4 commits into from
Nov 9, 2023

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Nov 7, 2023

Fixes #30947
Addresses #28161

Thanks @gregoryagu! 🎷 ... This needed a lot of work. I'm glad you called it out.

  • I'm adding a whole section based on what Steve wrote, and I'm cross-linking that in the Migration topic.
  • I fix up the existing example ...
    • They can use the Routes component with full interactive server rendering ... but at the cost of adopting it across the app. This approach fine for an example that neatly replaces the layout approach of <8.0 ... AND I place a NOTE there to cover the alternative with a root-level cascading value, which includes a code example.
    • We already had a section on using root-level cascading values.

Do you want to look all of this over before I ping Steve?


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/components/cascading-values-and-parameters.md ASP.NET Core Blazor cascading values and parameters
aspnetcore/blazor/components/render-modes.md ASP.NET Core Blazor render modes
aspnetcore/migration/70-80.md Migrate from ASP.NET Core 7.0 to 8.0

@guardrex guardrex self-assigned this Nov 7, 2023
@gregoryagu
Copy link

Looks good to me!

@guardrex
Copy link
Collaborator Author

guardrex commented Nov 7, 2023

Thanks, I'll send Steve a message and see if he has time to take a look.

UPDATE: Done! ... If he's busy tho, I'm going to go ahead and merge this. I'm 🏃😰 trying to get everything done and can't be held up too much right now. The PRs start stepping on each other 😈 under high doc churn if I don't get them in quickly, and the other doc authors might need to get into the Migration article.

@guardrex guardrex merged commit 77c9fec into main Nov 9, 2023
3 checks passed
@guardrex guardrex deleted the guardrex/blazor-cascading-values-update branch November 9, 2023 20:28
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.

Sample does not address the issue with CascadingParameters not working with RenderModeInteractiveServer
2 participants