Skip to content

Conversation

@SteveSandersonMS
Copy link
Member

Provides fixes for:

This tweaks the templates for Blazor, Angular, MVC, Razor Pages, and Identity. So if someone from each of those product areas could please take a look, that would be valuable!

@SteveSandersonMS SteveSandersonMS requested a review from a team as a code owner September 7, 2020 16:34
@SteveSandersonMS
Copy link
Member Author

See individual commit messages for details about what each change fixes.

@javiercn
Copy link
Member

javiercn commented Sep 7, 2020

LGTM, some of these fixes are applicable to other templates I imagine, isn't that the case? (Like the navmenu one). I'm not asking you to do anything, just checking in case we need a separate bug to deal with them.

@SteveSandersonMS
Copy link
Member Author

The navmenu fix only applies in the cases I’ve applied it, as far as I see.

@SteveSandersonMS SteveSandersonMS force-pushed the stevesa/accessibility-fixes branch from 4fa30f5 to bed8eaa Compare September 8, 2020 09:22
@SteveSandersonMS SteveSandersonMS added this to the 5.0.0-rc2 milestone Sep 8, 2020
@mkArtakMSFT mkArtakMSFT added the Servicing-consider Shiproom approval is required for the issue label Sep 8, 2020
@ghost
Copy link

ghost commented Sep 8, 2020

Hello human! Please make sure you've included the Shiproom Template in a comment or (preferably) the PR description. Also, make sure this PR is not marked as a draft and is ready-to-merge.

@mkArtakMSFT
Copy link
Contributor

mkArtakMSFT commented Sep 8, 2020

@Pilchie can we get this approved for RC2? Thanks!

@Pilchie Pilchie added the Servicing-approved Shiproom has approved the issue label Sep 8, 2020
@ghost ghost removed the Servicing-consider Shiproom approval is required for the issue label Sep 8, 2020
@Pilchie
Copy link
Member

Pilchie commented Sep 8, 2020

Approved for .NET 5 RC2.

@Pilchie Pilchie added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Sep 8, 2020
@mkArtakMSFT mkArtakMSFT merged commit ecc2ba1 into release/5.0-rc2 Sep 9, 2020
@mkArtakMSFT mkArtakMSFT deleted the stevesa/accessibility-fixes branch September 9, 2020 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates Servicing-approved Shiproom has approved the issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants