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

[release/7.0] Update spa-templates to ref new 7.0 branch #44134

Merged
merged 1 commit into from
Sep 24, 2022

Conversation

dougbu
Copy link
Member

@dougbu dougbu commented Sep 22, 2022

[release/7.0] Update spa-templates to ref new 7.0 branch

Description

  • just created release/7.0 in dotnet/spa-templates
    • use it here to enable main in spa-templates to get new branding
  • also use latest commit from spa-templates

Contains a fix for [C&AI Translation Feedback] - ASP.NET Core incomplete German translation. #43956 brings in fixes for the templates in dotnat/aspnetcore for that bug too (more on that when that loc handback PR is in main and ready for backporting).

Customer Impact

Without this fix, customer would see Target net7.0 instead of Ziel net7.0 when in a German locale and using one of the templates from Microsoft.DotNet.Web.Spa.ProjectTemplates.7.0.

Regression?

  • Yes
  • No

Microsoft.DotNet.Web.Spa.ProjectTemplates.6.0 only had translations for the name and description of the two templates from dotnet/spa-templates. This change here completes filling that gap.

Risk

  • High
  • Medium
  • Low

One-word changes in existing language-specific files 😃

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

- just created release/7.0 in dotnet/spa-templates
  - use it here to enable main in spa-templates to get new branding
- also use latest commit from spa-templates
  - contains a fix for [[C&AI Translation Feedback] - ASP.NET Core incomplete German translation](https://dev.azure.com/ceapex/CEINTL/_workitems/edit/710128)
@dougbu dougbu requested review from a team, wtgodbe and Pilchie as code owners September 22, 2022 22:37
@dougbu dougbu added Servicing-consider Shiproom approval is required for the issue feature-spa feature-templates labels Sep 22, 2022
@ghost
Copy link

ghost commented Sep 22, 2022

Hi @dougbu. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.

To learn more about how to prepare a servicing PR click here.

@dougbu dougbu added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Sep 22, 2022
@dougbu dougbu added this to the 7.0.0 milestone Sep 22, 2022
@dougbu
Copy link
Member Author

dougbu commented Sep 22, 2022

In case it's not obvious, the [C&AI Translation Feedback] - ASP.NET Core incomplete German translation fix is in the dotnet/spa-templates repo but that doesn't affect what we ship until the submodule is updated here.

@dougbu
Copy link
Member Author

dougbu commented Sep 23, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@dougbu dougbu added Servicing-approved Shiproom has approved the issue and removed Servicing-consider Shiproom approval is required for the issue labels Sep 24, 2022
@dougbu
Copy link
Member Author

dougbu commented Sep 24, 2022

Approved in email

@dougbu dougbu merged commit a2d223c into dotnet:release/7.0 Sep 24, 2022
@dougbu dougbu deleted the dougbu/branch.spa-templates branch September 24, 2022 01:37
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 feature-spa feature-templates Servicing-approved Shiproom has approved the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants