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

Add locale FR + minor EN improvments #32

Merged
merged 3 commits into from
Feb 25, 2017
Merged

Conversation

Vadorequest
Copy link
Contributor

No description provided.

Various fixes
Add translation for FR
"menu": {
"shareDatas": "Share Datas",
"shareDatas": "Share data",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI data doesn't take any "s" at the end, ever. ;)

},
"share": {
"text": "Wanna check your GitHub data analysis visually ?",
"text": "Wanna check your GitHub data analysis visually?",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI In english, punctuation is always close to the "left word".

@@ -0,0 +1,61 @@
{
"language": {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure what this language key is for, it seems it is mixed up in the zh-CN and en-US files

Copy link
Owner

Choose a reason for hiding this comment

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

If used for user to change languages, but not work now because we use more than tow languages. I'll change this after merged.

@Vadorequest Vadorequest mentioned this pull request Feb 24, 2017
@ecmadao
Copy link
Owner

ecmadao commented Feb 25, 2017

Great work!!!!
Merged.

@ecmadao ecmadao merged commit ac946e6 into ecmadao:master Feb 25, 2017
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.

None yet

2 participants