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

Bound field or property expression convention #32511

Merged
merged 1 commit into from
May 6, 2024

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented May 3, 2024

Fixes #30656

cc: @AntMaster7 ... I had a few minutes to whip something up, but I'm not necessarily keen on my example and text. I say that this would have an obvious use in creating custom validation components, which isn't something that the Blazor docs here cover. This is a bit of an advanced framework feature for custom scenarios. Can you look this over along with the sample components ...

... and see if you think this will at least get it covered until Mackinnon is free. He's buried in work for the next few months, and I don't want to disturb him unless absolutely necessary. I'm going to make a tracking entry to make sure that I discuss this section and example with him later. For now tho, do you think this is about what you would've needed to see for your own purposes back when you went looking for doc coverage on it?


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/components/data-binding.md ASP.NET Core Blazor data binding

@guardrex guardrex self-assigned this May 3, 2024
@guardrex
Copy link
Collaborator Author

guardrex commented May 6, 2024

I'm going to go ahead with this. Mackinnon will look at it later when his schedule opens up a bit.

@guardrex guardrex merged commit ae7e139 into main May 6, 2024
3 checks passed
@guardrex guardrex deleted the guardrex/bound-expression-tree branch May 6, 2024 15:01
@AntMaster7
Copy link

Thank you @guardrex. I haven't taken the time yet to look into it but the example looks good. I do not have permission however to open the preview link.

@guardrex
Copy link
Collaborator Author

guardrex commented May 6, 2024

Thanks! At least, this will provide a little guidance until Mackinnon can look. He may be able to provide better descriptive language.

I do not have permission however to open the preview link.

That's normal. It's for internal review by Microsoft folks.

@guardrex guardrex mentioned this pull request Jan 3, 2025
68 tasks
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.

Undocumented {PARAMETER NAME}Expression feature in blazor data binding documentation.
2 participants