Patch #33773
closedPatches to r19937 with update russian translation
0%
Description
Files:
- config/locales/ru.yml
- public/javascripts/jstoolbar/lang/jstoolbar-ru.js
Files
Updated by Go MAEDA over 4 years ago
Thank you for providing the Russian translation.
However, I cannot commit the patch now because the patch changes the existing three translations, field_due_date, field_estimated_hours, and text_convert_available. Since I don't understand Russian at all, I cannot determine whether the change is appropriate. It should be discussed by those who understand the Russian language.
If the patch doesn't include changes to the three strings, I will commit the patch quickly. Could you consider opening a new issue to separate the changes to the existing three translations and discuss them?
Updated by VVD VVD over 4 years ago
- File ru.yml.new.diff ru.yml.new.diff added
Attached new patch for new translations only.
Separate patch in #33774 for fix existing translations.
Updated by Vasili Korol over 4 years ago
I agree with most of the proposed translations but have a few remarks:
ru.yml.diff
:
field_esitmated_hours
: The change of "Оценка временных затрат" to "Оценка трудозатрат" is reverting the changes introduced in several older tickets (see, for example #26180). The translation of "estimated hours" as "трудозатраты" seems a bit ambiguous.error_bulk_download_size_too_big
: Is the translation of "in bulk" as "пакетно" conventional? I can also propose the following variant (sounds a bit more clear IMHO):
Эти файлы не могут быть загружены все разом, так как их общий объём превышает допустимый максимум (%{max_size})
setting_bulk_download_max_size
: The same - what if we change "пакетно" to "все разом"?
Максимальный общий объём для загрузки многих файлов сразу.
ru.yml.new.diff
:
error_bulk_download_size_too_big
: See abovesetting_bulk_download_max_size
: See above
Updated by VVD VVD over 4 years ago
I cannot discuss such things in English, so sorry for the Russian.
field_esitmated_hours:
Для этого создана отдельная задача: #33774
все разом
Это звучит как-то по деревенски… Совсем не как описание ошибки в ПО.
Тогда уж "одновременно".
Эти файлы не могут быть загружены одновременно, так как их общий объём превышает допустимый максимум (%{max_size})
Максимальный общий объём для загрузки многих файлов сразу.
А это звучит совсем не по русски.
Максимальный общий объём для одновременной загрузки
Максимальный общий объём при загрузке нескольких файлов
Updated by Vasili Korol over 4 years ago
I agree, this sounds better. As for field_estimated_hours
- I responded in #33774.
Updated by VVD VVD over 4 years ago
- File ru.yml.new.diff ru.yml.new.diff added
Максимальный суммарный объём при загрузке нескольких файлов
Updated by VVD VVD over 4 years ago
- File ru.yml.new.diff ru.yml.new.diff added
Sorry for noise.
error_bulk_download_size_too_big: Эти файлы не могут быть загружены одновременно,
так как их суммарный объём превышает допустимый максимум (%{max_size})
IMHO, word "суммарный" is better.
Updated by Vasili Korol over 4 years ago
yes.
So, the conclusion is:
error_bulk_download_size_too_big
:
Эти файлы не могут быть загружены одновременно, так как их суммарный объём превышает допустимый максимум (%{max_size})
setting_bulk_download_max_size
:
Максимальный суммарный объём для одновременной загрузки
Updated by Go MAEDA about 4 years ago
- File 33773-ru.patch 33773-ru.patch added
I have updated the patch for the latest trunk and changed some translations according to #33773#note-8.
VVD VVD and Vasili Korol, do you think 33773-ru.patch is OK to commit?
Updated by VVD VVD about 4 years ago
Typo:
error_attachments_too_many: Этот файл не будет загружен, так как прывышено максимальное
количество одновременно загружаемых файлов (%{max_number_of_files}}
Correct:
error_attachments_too_many: Этот файл не будет загружен, так как превышено максимальное
количество одновременно загружаемых файлов (%{max_number_of_files})
Other OK for me.
I see new lines:
setting_twofa: Two-factor authentication
label_optional: optional
label_required_lower: required
setting_twofa: Двухфакторная аутентификация
…
IMHO better create new issue - a lot of new records.
Updated by Vasili Korol about 4 years ago
Go MAEDA, the patch looks ok to me with the correction by VVD VVD. I also agree that the missing translations should be addressed by a new issue and a new patch.
Updated by Go MAEDA about 4 years ago
- File 33773-ru-v2.patch 33773-ru-v2.patch added
VVD VVD wrote:
Typo:
error_attachments_too_many: Этот файл не будет загружен, так как прывышено максимальное
количество одновременно загружаемых файлов (%{max_number_of_files}}Correct:
error_attachments_too_many: Этот файл не будет загружен, так как превышено максимальное
количество одновременно загружаемых файлов (%{max_number_of_files})Other OK for me.
Updated the patch. Thank you.
Updated by Go MAEDA about 4 years ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
Committed the patch. Thank you for your contribution.
Updated by VVD VVD about 4 years ago
What about this patch: jstoolbar-ru.js.diff?
Updated by Go MAEDA about 4 years ago
VVD VVD wrote:
What about this patch: jstoolbar-ru.js.diff?
Committed the patch. Thank you for pointing it out. I overlooked the patch.