Patch #33773

Patches to r19937 with update russian translation

Added by VVD VVD about 1 month ago. Updated 9 days ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:Translations
Target version:-

Description

Patches to r19937 with update russian translation.
Files:
  • config/locales/ru.yml
  • public/javascripts/jstoolbar/lang/jstoolbar-ru.js

ru.yml.diff Magnifier (4.76 KB) VVD VVD, 2020-07-27 11:27

jstoolbar-ru.js.diff Magnifier (1.02 KB) VVD VVD, 2020-07-27 11:31

ru.yml.new.diff Magnifier (3.57 KB) VVD VVD, 2020-07-27 17:07

ru.yml.new.diff Magnifier (3.61 KB) VVD VVD, 2020-07-28 22:39

ru.yml.new.diff Magnifier (3.62 KB) VVD VVD, 2020-07-28 22:40

33773-ru.patch Magnifier (3.8 KB) Go MAEDA, 2020-09-06 07:22

33773-ru-v2.patch Magnifier (3.8 KB) Go MAEDA, 2020-09-07 01:40

Associated revisions

Revision 20008
Added by Go MAEDA 11 days ago

Russian translation update (#33773).

Patch by VVD VVD.

Revision 20010
Added by Go MAEDA 9 days ago

Russian translation (jstoolbar-ru.js) update (#33773).

Patch by VVD VVD.

History

#1 Updated by Go MAEDA about 1 month 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?

#2 Updated by VVD VVD about 1 month ago

Attached new patch for new translations only.

Separate patch in #33774 for fix existing translations.

#3 Updated by Vasili Korol about 1 month 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 above
  • setting_bulk_download_max_size: See above

#4 Updated by VVD VVD about 1 month ago

I cannot discuss such things in English, so sorry for the Russian.

field_esitmated_hours:

Для этого создана отдельная задача: #33774

все разом

Это звучит как-то по деревенски… Совсем не как описание ошибки в ПО.
Тогда уж "одновременно".

Эти файлы не могут быть загружены одновременно, так как их общий объём превышает допустимый максимум (%{max_size})

Максимальный общий объём для загрузки многих файлов сразу.

А это звучит совсем не по русски.

Максимальный общий объём для одновременной загрузки

Максимальный общий объём при загрузке нескольких файлов

#5 Updated by Vasili Korol about 1 month ago

I agree, this sounds better. As for field_estimated_hours - I responded in #33774.

#6 Updated by VVD VVD about 1 month ago

Максимальный суммарный объём при загрузке нескольких файлов

#7 Updated by VVD VVD about 1 month ago

Sorry for noise.

error_bulk_download_size_too_big: Эти файлы не могут быть загружены одновременно,
так как их суммарный объём превышает допустимый максимум (%{max_size})

IMHO, word "суммарный" is better.

#8 Updated by Vasili Korol about 1 month ago

yes.

So, the conclusion is:

error_bulk_download_size_too_big:

Эти файлы не могут быть загружены одновременно, так как их суммарный объём превышает допустимый максимум (%{max_size})

setting_bulk_download_max_size:

Максимальный суммарный объём для одновременной загрузки

#9 Updated by VVD VVD 23 days ago

Commit last version?

#10 Updated by Go MAEDA 13 days ago

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?

#11 Updated by VVD VVD 13 days 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.

#12 Updated by Vasili Korol 13 days 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.

#13 Updated by Go MAEDA 12 days ago

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.

#14 Updated by Go MAEDA 11 days ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA

Committed the patch. Thank you for your contribution.

#15 Updated by VVD VVD 10 days ago

What about this patch: jstoolbar-ru.js.diff?

#16 Updated by Go MAEDA 9 days 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.

Also available in: Atom PDF