Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@emagnier
Copy link
Collaborator

@emagnier emagnier commented Dec 4, 2019

With the permission of @gtortone, I'm opening here a commit with his big work of translation ๐Ÿ‡ฎ๐Ÿ‡น.
Thanks @gtortone for this amazing work! ๐ŸŽ‰

This PR is open on the romcal-es6 branch, since there is currently some refactors in progress on locales and keys. It will be easier to take care of the new Italian locales in the same time.

I've also added a commit, to fix some duplicated keys. But this highlight that we need to find a more standard way to write them. @tukusejssirs you suggested somewhere to remove saint/blessed to simplify key names (that would make also reading easier in the locales files). It's something that we should consider and move forward on that.

@emagnier emagnier requested a review from tukusejssirs December 4, 2019 15:10
@gtortone
Copy link
Contributor

gtortone commented Dec 4, 2019

Hi @emagnier,
I just realized the problem of duplicated keys is true because I started from a translation that did not include these saints... Thanks !

@emagnier
Copy link
Collaborator Author

emagnier commented Dec 4, 2019

@gtortone you are right, new keys have been added in our working branch. No problem :) And thanks!

Copy link
Member

@tukusejssirs tukusejssirs left a comment

Choose a reason for hiding this comment

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

Good to go!

However, Iโ€™ve just suggested that we might want to replace ' with โ€™ (same would apply in other l10n files, e.g. fr.js).

@emagnier emagnier merged commit a7ea908 into romcal:romcal-es6 Dec 4, 2019
@emagnier emagnier deleted the italiano-es6 branch December 4, 2019 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants