Project

General

Profile

Actions

Patch #33773

closed

Patches to r19937 with update russian translation

Added by VVD VVD over 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Translations
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

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

Files

ru.yml.diff (4.76 KB) ru.yml.diff VVD VVD, 2020-07-27 11:27
jstoolbar-ru.js.diff (1.02 KB) jstoolbar-ru.js.diff VVD VVD, 2020-07-27 11:31
ru.yml.new.diff (3.57 KB) ru.yml.new.diff VVD VVD, 2020-07-27 17:07
ru.yml.new.diff (3.61 KB) ru.yml.new.diff VVD VVD, 2020-07-28 22:39
ru.yml.new.diff (3.62 KB) ru.yml.new.diff VVD VVD, 2020-07-28 22:40
33773-ru.patch (3.8 KB) 33773-ru.patch Go MAEDA, 2020-09-06 07:22
33773-ru-v2.patch (3.8 KB) 33773-ru-v2.patch Go MAEDA, 2020-09-07 01:40
Actions #1

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?

Actions #2

Updated by VVD VVD over 4 years ago

Attached new patch for new translations only.

Separate patch in #33774 for fix existing translations.

Actions #3

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 above
  • setting_bulk_download_max_size: See above
Actions #4

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})

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

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

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

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

Actions #5

Updated by Vasili Korol over 4 years ago

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

Actions #6

Updated by VVD VVD over 4 years ago

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

Actions #7

Updated by VVD VVD over 4 years ago

Sorry for noise.

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

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

Actions #8

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:

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

Actions #9

Updated by VVD VVD about 4 years ago

Commit last version?

Actions #10

Updated by Go MAEDA about 4 years 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?

Actions #11

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.

Actions #12

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.

Actions #13

Updated by Go MAEDA about 4 years 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.

Actions #14

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.

Actions #15

Updated by VVD VVD about 4 years ago

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

Actions #16

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.

Actions

Also available in: Atom PDF