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

chinese i18n tweaks #1523

Merged
merged 2 commits into from
Aug 19, 2021
Merged

chinese i18n tweaks #1523

merged 2 commits into from
Aug 19, 2021

Conversation

oshi97
Copy link
Contributor

@oshi97 oshi97 commented Aug 19, 2021

update edge supported locales for voice search (#1519)
rename zh-CN to zh-Hans and zh-TW to zh-Hant (#1517)

J=SLAP-1525
TEST=manual

ran build-languages for just chinese, assets had correct paths
Ran my codepen that brute force checks locales on edge, which works by
by turning on speech recognition for 10s, and adding the locale
to a list if a 'network' occurs. After 10s, the speech recognition
is aborted and a 'no-speech' error is expected.

Also added a zh-cn and zh-tw to the supported locales list, even
though they're not currently locales the SDK has built in, in case
a user manually supplies these locales.

J=SLAP-1532
TEST=manual,auto

manually test a number of the locales (~a third of them)
@oshi97 oshi97 merged commit fdfd847 into develop Aug 19, 2021
@oshi97 oshi97 deleted the feature/chinese-i18n branch August 19, 2021 14:49
@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 58.947% when pulling c42509c on feature/chinese-i18n into ac8a6b8 on develop.

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.

None yet

2 participants