Skip to content

Convert it translations from solidus 1.2 -> 1.3#56

Merged
jhawthorn merged 2 commits into
solidusio:masterfrom
luukveenis:update-it-translations
May 24, 2017
Merged

Convert it translations from solidus 1.2 -> 1.3#56
jhawthorn merged 2 commits into
solidusio:masterfrom
luukveenis:update-it-translations

Conversation

@luukveenis

Copy link
Copy Markdown
Contributor

Recent changes made to admin translations in solidus moved many of the keys. This was done to better use the ActiveModel translation conventions.

I wrote a script that scans through the locale files in solidus_i18n looking for missing keys when compared to en.yml in core. Since these translations are just moved, the script attempts to make the same moves in this locale as were made for english.

Reviews would be appreciated to find any blatant mistranslations.

@luukveenis luukveenis force-pushed the update-it-translations branch from 1f3f025 to 8a88084 Compare May 10, 2016 16:33
Comment thread config/locales/it.yml Outdated
iso_name: Nome ISO
name: Nome
numcode: Codice ISO
states_required: Stati Necessari

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be "Provincia Necessaria"

@tvdeyen

tvdeyen commented May 17, 2016

Copy link
Copy Markdown
Member

@mtylty and @alepore it would be very great, if you open a pr against @luukveenis branch, so we can merge this into it. Thanks for your help.

@mtylty

mtylty commented May 17, 2016

Copy link
Copy Markdown
Contributor

@tvdeyen it will be done!

Comment thread config/locales/it.yml
default_tax: Zona di Tassazione di Default
spree/adjustment:
adjustable: Variabile
amount: Quantità

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Quantità" means quantity but I think in this case "amount" is used to specify the price, maybe is better to use "Prezzo".
@alepore @mtylty what do you think?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch. this probably needs to be checked in the admin...

@tvdeyen

tvdeyen commented Jul 7, 2016

Copy link
Copy Markdown
Member

@AlessioRocco @mtylty @alepore could you please have a look at #62 ? Thanks

@tvdeyen

tvdeyen commented Feb 7, 2017

Copy link
Copy Markdown
Member

@luukveenis any new on this?

@luukveenis luukveenis force-pushed the update-it-translations branch from d11085e to 2b8b5ee Compare May 23, 2017 15:25
@jhawthorn jhawthorn merged commit 6898def into solidusio:master May 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants