Skip to content

Enhance MEV documentation#37320

Open
Youssef1313 wants to merge 2 commits into
mainfrom
dev/ygerges/val
Open

Enhance MEV documentation#37320
Youssef1313 wants to merge 2 commits into
mainfrom
dev/ygerges/val

Conversation

@Youssef1313

@Youssef1313 Youssef1313 commented Jul 8, 2026

Copy link
Copy Markdown
Member

I don't think the previous documentation was done in the correct places.

MVC doesn't have built-in support for Microsoft.Extensions.Validation AFAIK.

This PR removes MEV from MVC docs, and creates a dedicated page for MEV under minimal API documentation.


Internal previews

📄 File 🔗 Preview link
aspnetcore/fundamentals/minimal-apis/validation.md customer intent: As an ASP.NET developer, I want to have automatic validation of models in Minimal API parameters.
aspnetcore/mvc/models/validation.md Model validation in ASP.NET Core MVC
aspnetcore/toc.yml aspnetcore/toc
aspnetcore/tutorials/first-mvc-app/validation.md Part 9, add validation to an ASP.NET Core MVC app

Comment thread aspnetcore/fundamentals/minimal-apis/validation.md
Comment thread aspnetcore/fundamentals/minimal-apis/validation.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/validation.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/validation.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/validation.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/validation.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/validation.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/validation.md Outdated
Comment thread aspnetcore/fundamentals/minimal-apis/validation.md Outdated
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
@Youssef1313 Youssef1313 requested a review from oroztocil July 8, 2026 19:55

@wadepickett wadepickett left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Youssef1313, Looks great! Thanks for fixing this. I made a few suggestions inline for you to consider.

@Youssef1313

Copy link
Copy Markdown
Member Author

@wadepickett The previously posted comments were already addressed. Are there any further suggestions?

@wadepickett

wadepickett commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@wadepickett The previously posted comments were already addressed. Are there any further suggestions?

Looks great. Sorry, I should have looked at the refresh of the page first. You are moving quick. I did add a follow up question on the package above. Is an implicit PackageReference to Microsoft.Extensions.Validation added when an ASP.NET Core project is built or do we need a line of instruction on that?

@wadepickett wadepickett left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Youssef1313, Approved. I checked on the implicit package reference for v10 and it was there. Sorry for the confusion. Great work, thanks again for this article.

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.

2 participants