Skip to content

Conversation

@guardrex
Copy link
Collaborator

@guardrex guardrex commented Nov 14, 2024

Fixes #34127

Thanks @willdean! 🚀 ... I'll hold it for sec while you check out the update. I'm not going to wait for the product unit on this one. I agree with your suggestion for the change.


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/components/data-binding.md aspnetcore/blazor/components/data-binding

@willdean
Copy link

LGTM (the Preview Link isn't visible to outsiders, but it's clear enough in the .md)

Thx

@guardrex
Copy link
Collaborator Author

Correct ... that link is for internal review. I asked management several years ago if they would open it up for everyone, but I got crickets 🦗 from them 😆.

This PR will probably go live by EOD.

I'll get to your other issue early next week. That one is a bit more nuanced. I'd like Mackinnon (at least) to chime in on if they prefer mentioning it in passing or actually changing the content/code to more forcefully recommend trapping JSDisconnectedException.

@guardrex guardrex merged commit c830a92 into main Nov 14, 2024
3 checks passed
@guardrex guardrex deleted the guardrex-patch-5 branch November 14, 2024 18:37
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.

Blazor bind:after description is incorrect on the "Blazor Data binding" page.

3 participants