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

AtlasEngine: Fix han unification issues #15358

Merged
merged 1 commit into from
May 15, 2023
Merged

Conversation

lhecker
Copy link
Member

@lhecker lhecker commented May 15, 2023

This commit ensures that we pass the user's locale to MapCharacters.

Validation Steps Performed

See: https://heistak.github.io/your-code-displays-japanese-wrong/
After modifying the userLocaleName to contain ja-JP, zh-CN and
zh-TW, printing "刃直海角骨入" produces the expected, localized result.

@lhecker lhecker added Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-1 A description (P1) Area-AtlasEngine labels May 15, 2023
@lhecker lhecker merged commit 4c3d3d8 into main May 15, 2023
@lhecker lhecker deleted the dev/lhecker/atlas-engine-fixup2 branch May 15, 2023 18:40
DHowett pushed a commit that referenced this pull request May 15, 2023
This commit ensures that we pass the user's locale to `MapCharacters`.

## Validation Steps Performed
See: https://heistak.github.io/your-code-displays-japanese-wrong/
After modifying the `userLocaleName` to contain `ja-JP`, `zh-CN` and
`zh-TW`, printing "刃直海角骨入" produces the expected, localized result.

(cherry picked from commit 4c3d3d8)
Service-Card-Id: 89211793
Service-Version: 1.18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-AtlasEngine Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-1 A description (P1)
Projects
Development

Successfully merging this pull request may close these issues.

3 participants