Patch #32326

Update it.yml locale

Added by pasquale [:dedalus] 11 months ago. Updated 11 months ago.

Status:Needs feedbackStart date:
Priority:NormalDue date:
Assignee:pasquale [:dedalus]% Done:

0%

Category:Translations
Target version:-

Description

  • added various missing strings;
  • reorder strings into the file;
  • made some changes to already translated strings for consistency.

001-it-locale.diff Magnifier - apply to trunk (82.2 KB) pasquale [:dedalus], 2019-10-23 13:22

001-it-locale-v0.2.diff Magnifier (82.2 KB) pasquale [:dedalus], 2019-10-23 16:00

002-02-adds missing key-for-label_show_closed_project.diff Magnifier (424 Bytes) pasquale [:dedalus], 2019-10-24 18:11

002-03-fix-some-string-capitalization.diff Magnifier (1.17 KB) pasquale [:dedalus], 2019-10-24 18:11

002-04-fix-incomplete-translation-for-notice_successful-keys.diff Magnifier (1.26 KB) pasquale [:dedalus], 2019-10-24 18:11

002-05-cumulative-patch-to-fix-existing-translations.diff Magnifier (6.1 KB) pasquale [:dedalus], 2019-10-24 18:11

002-01-adds-missing-translations.diff Magnifier (13.2 KB) pasquale [:dedalus], 2019-10-24 18:11

rr.png (65.7 KB) pasquale [:dedalus], 2019-10-27 09:47


Related issues

Related to Redmine - Defect #32354: Fix inconsistent capitalization in Italian translation Closed
Related to Redmine - Patch #32358: Fix incomplete Italian translation for notice_successful ... Closed
Related to Redmine - Patch #32380: Change Italian translation for "news" Closed

Associated revisions

Revision 18864
Added by Go MAEDA 11 months ago

Italian translation update (#32326).

Patch by pasquale [:dedalus].

History

#1 Updated by pasquale [:dedalus] 11 months ago

Adds to previous patch:

  • remove trailing whitespaces from my previous patch;
  • adds missing string label_show_closed_projects that is missing in config\locale\it.yml in redmine 4.0.5

The new patch attached is a replacement of the previous one (the previous one is now obsolete)

#2 Updated by Go MAEDA 11 months ago

Thank you for working hard on updating it.yml.

  • Do you have a translation file without reordering? Reordering makes it difficult to compare the current and the new translation
  • Could you split the patch into a missing translations patch and a changing existing translation patch? I will commit a patch for missing translations quickly but would not commit changes to existing translation because I don't understand Italian and I cannot judge the changes are appropriate

#3 Updated by pasquale [:dedalus] 11 months ago

Go MAEDA wrote:

Thank you for working hard on updating it.yml.

  • Do you have a translation file without reordering? Reordering makes it difficult to compare the current and the new translation
  • Could you split the patch into a missing translations patch and a changing existing translation patch? I will commit a patch for missing translations quickly but would not commit changes to existing translation because I don't understand Italian and I cannot judge the changes are appropriate
Here a new set of patches that are a replacement of previous ones:

Thanks in advance.

#4 Updated by Go MAEDA 11 months ago

Thank you for splitting the patch. It is really helpful for me.

I looked 002-01-adds-missing-translations.diff and found that the translation for label_relations is "Relationi", but other existing translation for "relation" is "Relazioni". Is "Relationi" correct?

  label_relations: Relationi

#5 Updated by pasquale [:dedalus] 11 months ago

Go MAEDA wrote:

Is "Relationi" correct?

No, sorry it is a typo error: the right word Is "Relazioni"

#6 Updated by Go MAEDA 11 months ago

  • Related to Defect #32354: Fix inconsistent capitalization in Italian translation added

#7 Updated by Go MAEDA 11 months ago

I have just opened #32354 to handle the patch 002-03-fix-some-string-capitalization.diff.

#8 Updated by Go MAEDA 11 months ago

  • Category changed from I18n to Translations

#9 Updated by pasquale [:dedalus] 11 months ago

Hi Go MAEDA, thanks for reviewing my patches.
Maybe a clarification is needed: the patchattachment:"002-02-adds missing key-for-label_show_closed_project.diff" fix an issue on the official it.yml file where the line related at the key label_show_closed_project is missing, so the string "View closed projects" cannot be translate at all.
Thanks!

#10 Updated by Go MAEDA 11 months ago

pasquale [:dedalus] wrote:

Maybe a clarification is needed: the patchattachment:"002-02-adds missing key-for-label_show_closed_project.diff" fix an issue on the official it.yml file where the line related at the key label_show_closed_project is missing, so the string "View closed projects" cannot be translate at all.

The latest trunk no longer uses label_show_closed_project, so the key has been removed in r18785 (#29482).

#11 Updated by pasquale [:dedalus] 11 months ago

Go MAEDA wrote:

The latest trunk no longer uses label_show_closed_project, so the key has been removed in r18785 (#29482).

Ah ok sorry, I have missed that!

#12 Updated by Go MAEDA 11 months ago

  • Related to Patch #32358: Fix incomplete Italian translation for notice_successful keys added

#13 Updated by Go MAEDA 11 months ago

  • Status changed from New to Needs feedback
  • Assignee set to pasquale [:dedalus]

Hello pasquale [:dedalus], your translation patch (002-05-cumulative-patch-to-fix-existing-translations.diff) changes the Italian words "notizia" and "sottoattività" to English word "news" and "subtask". Could you clarify the reason?

#14 Updated by pasquale [:dedalus] 11 months ago

Go MAEDA wrote:

Hello pasquale [:dedalus], your translation patch (002-05-cumulative-patch-to-fix-existing-translations.diff) changes the Italian words "notizia" and "sottoattività" to English word "news" and "subtask". Could you clarify the reason?

For news instead of literal translation notizia, because in current language the english word is used and the literal translation is used rarely (xref http://www.treccani.it/vocabolario/news/)
For subtask instead of literal translation sottoattività, because I believe that the english term is more appropriate in a agile context because in current language we refer to task\subtask and not to attività\sottoattività (xref http://www.treccani.it/vocabolario/task/)

#15 Updated by Go MAEDA 11 months ago

  • Related to Patch #32380: Change Italian translation for "news" added

Also available in: Atom PDF