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

I18n #15

Open
Vadorequest opened this issue Feb 21, 2017 · 17 comments
Open

I18n #15

Vadorequest opened this issue Feb 21, 2017 · 17 comments

Comments

@Vadorequest
Copy link
Contributor

I looked quickly in the repo and couldn't find any translation file. Also, it seems hard-coded.
Could you extract it in a dedicated translation file? I'd basically add support for French/English languages.

For the record, your repo is top 10 trending GitHub projects today! Cheers :)

https://github.com/trending/javascript

@ecmadao
Copy link
Owner

ecmadao commented Feb 21, 2017

@Vadorequest
Wow coooooooooooooool! I even didn't think of that before.
Actually, this website was designed for Chinese developers at first, but I'm also interested in supporting multiple languages. I'll work on that these days.
Thank you!

@ecmadao
Copy link
Owner

ecmadao commented Feb 22, 2017

Hey @Vadorequest
Glad to say that I have support a basic English version now 😁 --> http://hacknical.com
Atleast, github data analysis page totally support EN, and I'll working on resume page later.
Thanks again !

@Vadorequest
Copy link
Contributor Author

Vadorequest commented Feb 22, 2017 via email

@Vadorequest
Copy link
Contributor Author

#23

@Vadorequest
Copy link
Contributor Author

Vadorequest commented Feb 22, 2017

@ecmadao PR done! (english only yet, I'll add FR when we're good with EN)

I looked at the website, almost everything is translated now, thanks!

I shared you website on Slack and I got a mix of English/Chinese in the preview mode:

image

I guess this part has been missed ;)

@ecmadao
Copy link
Owner

ecmadao commented Feb 22, 2017

@Vadorequest
Pretty cool. I'm quite sure that I have few mistakes on translation, glad to see you are working on them. THX!

@ecmadao
Copy link
Owner

ecmadao commented Feb 23, 2017

@Vadorequest
Almost Done 😁
You can check and review now.

@Vadorequest
Copy link
Contributor Author

Vadorequest commented Feb 24, 2017

@ecmadao FR version added + minor EN review

#32

@Vadorequest
Copy link
Contributor Author

Also, in the graphics there are some missing translation too :)

image

@ecmadao
Copy link
Owner

ecmadao commented Feb 25, 2017

@Vadorequest

The bug of texts in chart has fixed now, and FR version is initialized now, cool!!! I'm really appreciate of your contributions.

However, if we wanna totally support FR, we also have to check locale files at frontend locales folder, then add new file names fr.js in each folder. Besides, you can find and check EN locale files there.

As for me, I will working on fetching orgs info for each user, and analyse users contributions for each forked & org repository.

@Vadorequest
Copy link
Contributor Author

Cool! Yep, I've missed the frontend part ^^
I'll take a look ASAP :)

@Vadorequest
Copy link
Contributor Author

@ecmadao I'll use files with the name fr.js and not fr-FR.js, following the same pattern as the en.js. Good for you?

@Vadorequest
Copy link
Contributor Author

A few questions:

  • I wonder what you mean by PV?
  • I didn't understood until now that Resume Share Datas / GitHub Share Datas were "tabs" I thought they were just text, not links, maybe it should be made more obvious?
  • About the Platform and Browser, "Max Platform", don't you mean "Preferred platform" instead? I guess you can somehow figure out what's the platform/browser GitHub users use? I didn't know it was possible.

@Vadorequest
Copy link
Contributor Author

@ecmadao New PR :)

#35

@Vadorequest
Copy link
Contributor Author

But I think the english version must be reviewed by a chinese-english speaker first, it's not really easy to translate as it, and it feels like the choice of the words can be improved ;)

Note that the french version is not fully covered yet, sometimes I don't really how I should translate ^^

@ecmadao
Copy link
Owner

ecmadao commented Feb 27, 2017

@Vadorequest

Use fr.js instead of fr-FR.js is cool.😁

PV means Page view, and Max Platform is what you think, I record each visit to the shared link, and figure out most preferred platform visitors used. It's my fault to not translate it clearly.

Besides, thanks for your advance of Resume Share Datas / GitHub Share Datas tab, I'll find a better way to show them.
Soooo sorry for my pool English transition that bothered you.😵

@Vadorequest
Copy link
Contributor Author

@ecmadao It's fine, don't worry ;) I took a look at your other repo and I also like the GitHub extension!
Besides, I sucked at English for so long... If we have had that discussion 5 years ago you'd have looked like Shakespeare to me! ^_^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants