-
Notifications
You must be signed in to change notification settings - Fork 25k
Document validation models from different assemblies #36246
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 adds documentation for validating forms with models defined in different assemblies, a new feature in ASP.NET Core 10.0. The changes guide users on setting up validation for models in libraries or .Client projects of Blazor Web Apps.
- Adds a new section explaining how to validate models from external assemblies
- Provides a code example showing service collection extension setup
- Includes a reference to the detailed documentation in the release notes
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| aspnetcore/release-notes/aspnetcore-10/includes/blazor.md | Adds a brief section with cross-reference to the detailed validation documentation |
| aspnetcore/blazor/forms/validation.md | Adds comprehensive documentation with explanation, requirements, and code example for cross-assembly validation |
Co-authored-by: Copilot <[email protected]>
oroztocil
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.
Looks great!
|
I had one final little NIT there. I didn't remind devs that they need the namespace, so I added a remark and a line of code on that. Ok ... we're good here. I'll merge this right after it finishes building. |
Fixes #36240
Ready! 🎉 ... I derived this from the PU issue discussion with a bit of code help from Copilot.
Internal previews