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

Error: Couldn't find translations for ru-RU #3882

Closed
alekseyr1987 opened this issue Apr 18, 2020 · 9 comments · Fixed by #3904
Closed

Error: Couldn't find translations for ru-RU #3882

alekseyr1987 opened this issue Apr 18, 2020 · 9 comments · Fixed by #3904

Comments

@alekseyr1987
Copy link
Contributor

Hi everyone!

Screenshot at Apr 18 17-58-48

@JohnMcLear
Copy link
Member

image

@JohnMcLear
Copy link
Member

file name is ru.json

@JohnMcLear
Copy link
Member

Possibly related to #3404 ?

@alekseyr1987
Copy link
Contributor Author

Browser - Chrome
Browser language - Russian
HTTP_ACCEPT_LANGUAGE - ru-RU,ru;q=0.9,en-US;q=0.8,en;q=0.7

and

Screenshot at Apr 18 22-49-01

@alekseyr1987
Copy link
Contributor Author

Browser - Firefox
Browser language - English
HTTP_ACCEPT_LANGUAGE - en,de;q=0.7,en-US;q=0.3

all is ok

@JohnMcLear
Copy link
Member

JohnMcLear commented Apr 18, 2020

Yea, I mentioned that if ru-RU file is not available it should always fall back to language.substring(0,1)+".json", it's an easy fix I have put someone on it.

@iquidus
Copy link
Contributor

iquidus commented Apr 19, 2020

Having some trouble reproducing this.

Related: https://github.com/ether/etherpad-lite/blob/develop/src/static/js/html10n.js#L186

@JohnMcLear
Copy link
Member

JohnMcLear commented Apr 19, 2020 via email

@iquidus
Copy link
Contributor

iquidus commented Apr 19, 2020

ok, i am able to replicate in firefox using ru-MD on the home page. The issue appears to go a little deeper than originally thought. ru-RU -> ru is being handled correctly as is (by line 186 linked above). However its then getting set back to en by this line here

https://github.com/ether/etherpad-lite/blob/develop/src/static/js/l10n.js#L7

studying further.

iquidus added a commit to iquidus/etherpad-lite that referenced this issue Apr 19, 2020
iquidus added a commit to iquidus/etherpad-lite that referenced this issue Apr 19, 2020
muxator pushed a commit to iquidus/etherpad-lite that referenced this issue Apr 21, 2020
muxator pushed a commit to iquidus/etherpad-lite that referenced this issue Apr 21, 2020
muxator pushed a commit to iquidus/etherpad-lite that referenced this issue Apr 26, 2020
…ble locales

Before this change, we simply generated an error.

For example:
- if the browser sent 'ru-RU', but Etherpad has 'ru' available, select 'ru';
- if the browser sent 'zh', but we have 'zh-hans' available, use 'zh-hans'.

Fixes ether#3882.
muxator pushed a commit that referenced this issue Apr 26, 2020
…ble locales

Before this change, we simply generated an error.

For example:
- if the browser sent 'ru-RU', but Etherpad has 'ru' available, select 'ru';
- if the browser sent 'zh', but we have 'zh-hans' available, use 'zh-hans'.

Fixes #3882.
winstonsung added a commit to winstonsung/etherpad-lite that referenced this issue Feb 8, 2024
Bug: ether#6129
Follows-Up: ether#3882
Change-Id: Ie3fcec84948fde36fa12448d5eabaf05f79f1283
winstonsung added a commit to winstonsung/etherpad-lite that referenced this issue Feb 8, 2024
fixes ether#6129

Bug: ether#6129
Bug: ether#6144
Follows-Up: ether#3882
Change-Id: Ie3fcec84948fde36fa12448d5eabaf05f79f1283
winstonsung added a commit to winstonsung/etherpad-lite that referenced this issue Feb 8, 2024
fixes ether#6129

Bug: ether#6129
Bug: ether#6144
Follows-Up: ether#3882
Change-Id: Ie3fcec84948fde36fa12448d5eabaf05f79f1283
winstonsung added a commit to winstonsung/etherpad-lite that referenced this issue Feb 8, 2024
fixes ether#6129

Bug: ether#6129
Bug: ether#6144
Follows-Up: ether#3882
Change-Id: Ie3fcec84948fde36fa12448d5eabaf05f79f1283
SamTV12345 pushed a commit that referenced this issue Feb 10, 2024
fixes #6129

Bug: #6129
Bug: #6144
Follows-Up: #3882
Change-Id: Ie3fcec84948fde36fa12448d5eabaf05f79f1283
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants