Supports configuring OpenSearch Dashboards to zh-CN, zh-TW, tr-TR, pt-PT, pt-BR, ko-KR, ja-JP, it-IT, id-ID, fr-FR, fr-CA, es-ES, es-419, de-DE - #12709
Conversation
PR Reviewer Guide 🔍(Review updated until commit b210fcc)Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Latest suggestions up to b210fcc
Previous suggestionsSuggestions up to commit 6628d99
|
…R`, `pt-PT`, `pt-BR`, `ko-KR`, `ja-JP`, `it-IT`, `id-ID`, `fr-FR`, `fr-CA`, `es-ES`, `es-419`, `de-DE` Signed-off-by: Qiheng He <linghengqian@outlook.com>
6628d99 to
b210fcc
Compare
|
Persistent review updated to latest commit b210fcc |
❌ 13 Jest Test Failure(s)📄
|
linghengqian
left a comment
There was a problem hiding this comment.
Since the failed CIs are all due to Exceeded timeout of 30000 ms for a hook, this doesn't seem to be related to my changes. The main branch already has Flaky tests.
Description
zh-CN,zh-TW,tr-TR,pt-PT,pt-BR,ko-KR,ja-JP,it-IT,id-ID,fr-FR,fr-CA,es-ES,es-419,de-DE.dot: falseexcludes dotfiles fromsrc/**, so the i18n registration manifest must be listed explicitly. Without it the distributable shipssrc/translations/*.jsonbut the server's*/.i18nrc.jsonlookup finds zero translation paths, soi18n.locale(e.g.I18N_LOCALE=zh-CNin the Docker image) silently stays in English. See [BUG] Docker Image'sI18N_LOCALEenvironment variable does not seem to work #8075 (comment) .Issues Resolved
I18N_LOCALEenvironment variable does not seem to work #8075 .Screenshot
Testing the changes
I18N_LOCALEenvironment variable does not seem to work #8075 (comment) .Check List
yarn test:jestyarn test:jest_integration