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

Update ukrainian translation #5405

Merged
merged 5 commits into from
May 8, 2024
Merged

Update ukrainian translation #5405

merged 5 commits into from
May 8, 2024

Conversation

novashdima
Copy link
Contributor

  • Remove deleted ua_UA, ua
  • Remove invalid ru_UA
  • Update uk-UA, uk

@xet7
Copy link
Member

xet7 commented May 7, 2024

@novashdima

Translations are done at https://app.transifex.com/wekan/wekan , not as pull requests.

Only, when adding new features, new strings are added to en.i18n.json as pull request.

@xet7 xet7 closed this May 7, 2024
@novashdima
Copy link
Contributor Author

@xet7 ok
and how to remove obsolete and invalid translations?

@xet7
Copy link
Member

xet7 commented May 8, 2024

@novashdima

In what way they are obsolete or invalid?

@xet7
Copy link
Member

xet7 commented May 8, 2024

@novashdima

Those language options exist at Transifex. Sometime, someone has added translations for some languages. I don't usually remove translations, because I don't know what people where speak that language. I don't know can any language become obsolete.

@novashdima
Copy link
Contributor Author

@xet7

The current name for the Ukrainian language in the iso is uk_UA (Ukraine Ukrainian), ua_UA and ru_UA are old names for the Ukrainian language.
Also, the translation files ua and ua_UA have been deleted, but they exist in the configuration. Therefore, I see 5 Ukrainian languages in the application, of which ua and ua_UA cannot be selected because there are no localization files, ru_UA is an outdated name for Ukrainian, and the localization file is in Russian.

uk and uk_UA are the correct names for Ukrainian, they should be left, I will update them via the app you threw down.

@xet7 xet7 reopened this May 8, 2024
@xet7 xet7 merged commit a3d4c3d into wekan:main May 8, 2024
7 checks passed
@xet7
Copy link
Member

xet7 commented May 8, 2024

Hmm, just a moment, I try to update Transifex directly.

@novashdima
Copy link
Contributor Author

@xet7

I already update most of the lines manually

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