Skip to content

Light Freshness Edit: ASP.NET - mvc - #37085

Merged
wadepickett merged 5 commits into
dotnet:mainfrom
lootle1:FR50
May 4, 2026
Merged

Light Freshness Edit: ASP.NET - mvc#37085
wadepickett merged 5 commits into
dotnet:mainfrom
lootle1:FR50

Conversation

@lootle1

@lootle1 lootle1 commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

This PR contains freshness updates to the following articles:

  • Overview of ASP.NET Core MVC
    [Answered by Wade in other now closed PR here pushed to this PR] Row 119, the link https://jqueryvalidation.org/ goes to a page with an invalid SSL and host error. Perhaps they're in the process of updating but curious if you want to remove, replace, or leave as is? I did search for a replacement in case we want to replace but didn't locate anything.
  • Layout in ASP.NET Core
  • Views in ASP.NET Core MVC
  • Razor syntax reference for ASP.NET Core
  • Partial views in ASP.NET Core

Minor grammar/style updates. Link updates.
For request details, see 568764


Internal previews

📄 File 🔗 Preview link
aspnetcore/mvc/overview.md Overview of ASP.NET Core MVC
aspnetcore/mvc/views/layout.md Layout in ASP.NET Core
aspnetcore/mvc/views/overview.md Views in ASP.NET Core MVC
aspnetcore/mvc/views/partial.md Partial views in ASP.NET Core
aspnetcore/mvc/views/razor.md aspnetcore/mvc/views/razor

@lootle1
lootle1 marked this pull request as ready for review April 30, 2026 21:24
@lootle1

lootle1 commented Apr 30, 2026

Copy link
Copy Markdown
Contributor Author

@tdykstra @wadepickett this PR is ready for review. Top comment has an overview. Note I ran into an issue on the previous PR (also in top comment) so pushed all those changes here along with the new ones.

Let me know if any additional change requests, thank you!

Comment thread aspnetcore/mvc/views/overview.md Outdated
Comment thread aspnetcore/mvc/views/overview.md Outdated
Comment thread aspnetcore/mvc/overview.md Outdated
Comment thread aspnetcore/mvc/views/razor.md Outdated

@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.

Looks great! I found a few small item and one medium one to fix before we merge.

lootle1 and others added 4 commits May 4, 2026 08:39
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
@lootle1

lootle1 commented May 4, 2026

Copy link
Copy Markdown
Contributor Author

Thanks @wadepickett, your changes looks great to me and have committed. This PR is all set from my vantage point!

@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.

Approved. Looks good, thanks!

@wadepickett
wadepickett merged commit a32f16c into dotnet:main May 4, 2026
4 checks passed
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