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

fix: Correcting language override to use full culture name and added Localization page to TestHarness #725

Merged
merged 2 commits into from
Sep 19, 2022

Conversation

nickrandolph
Copy link
Contributor

@nickrandolph nickrandolph commented Sep 13, 2022

closes: #721

PR Type

What kind of change does this PR introduce?

  • Bugfix

What is the current behavior?

Two character language code used instead of Culture name
PrimaryLanguageOverride set too late in app startup sequence

What is the new behavior?

Added a new page to the TestHarness for Localization
PrimaryLanguageOverride is set in a service initialization method immediately after the IHost has been created

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested code with current supported SDKs
  • Docs have been added/updated which fit documentation template. (for bug fixes / features)
  • Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
  • Contains NO breaking changes
  • Updated the Release Notes
  • Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

@github-actions
Copy link

github-actions bot commented Sep 19, 2022

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-moss-0c5b8040f-725.eastus2.azurestaticapps.net

1 similar comment
@github-actions
Copy link

github-actions bot commented Sep 19, 2022

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-moss-0c5b8040f-725.eastus2.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Sep 19, 2022

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-moss-0c5b8040f-725.eastus2.azurestaticapps.net

1 similar comment
@github-actions
Copy link

github-actions bot commented Sep 19, 2022

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-moss-0c5b8040f-725.eastus2.azurestaticapps.net

@nickrandolph nickrandolph merged commit 8979a07 into main Sep 19, 2022
@nickrandolph nickrandolph deleted the dev/nr/localizationtesting branch September 19, 2022 12:27
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.

[Localization] Language changes won't apply
3 participants