Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert it translations from solidus 1.2 -> 1.3 #56

Merged
merged 2 commits into from
May 24, 2017

Conversation

luukveenis
Copy link
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.

@@ -23,6 +25,7 @@ it:
iso_name: Nome ISO
name: Nome
numcode: Codice ISO
states_required: Stati Necessari
Copy link
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
Copy link
Member

tvdeyen commented May 17, 2016

@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
Copy link
Contributor

mtylty commented May 17, 2016

@tvdeyen it will be done!

default_tax: Zona di Tassazione di Default
spree/adjustment:
adjustable: Variabile
amount: Quantità

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
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
Copy link
Member

tvdeyen commented Jul 7, 2016

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

@tvdeyen
Copy link
Member

tvdeyen commented Feb 7, 2017

@luukveenis any new on this?

@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