I'm more then open to pull requests! Make sure to check out my Zenhub board so we don't have any conflicts if I or someone else has already started on an issue.
JSDocs for the majority of functions can be found here.
Rules
- Everything is in ES6 Javascript with the Node 4 presets. That means
constandlet, everything for example. - New features must be discussed in an issue first before a PR, this prevents wasting anyones time incase it's already being worked on.
- Reframe from adding new npm modules if there's already something that'll get the job done.
- Must pass all tests.
npm test - Must write tests for all new functions (look at current tests for examples, still a WIP).
- Functions must be camel cased.
downloadStuff() - Variables must be underscored.
my_champion = 'teemo' - PR must be off and back to master branch.
I suggest making sure your linters are setup correctly, or run npm run lint every now and again so you're not making a crazy amount of commits.
Adding A Source
You can find examples for adding sources in src/sources. Each source requires an object export of source_info, a function called getSr() for summoners rift item sets, and getVerion() for the current LoL patch version. Once finished, the app will automatically add the new source to sources list. Make sure to add a nice little 16x16 icon in the app/img/sources folder. For an easy example check out src/sources/koreanbuilds.js.
I use Transifex to manage all my translations, it's super easy and lets everyone contribute! The initial translations were done with Google Translate, so just because a translation exists doesn't mean it's correct. You can also check out the language status list below to see which have been done by Google, and which has had attention from a native speaker.
Rules
- Make sure to check if a translation has been done already before writing it over again.
- If you have an issue with someone elses translation, please use the comments section.
- Don't add extra punctuations (such as
...) to the end of translations if the English version doesn't have it.
Setup
- Create an account with Transifex (Or login with Github/Google/Facebook/ect)
- Join the Championify team with the language you'd like to work on
- Start!
Language Status
| Language | Finished |
|---|---|
| English | Yes |
| Arabic | Yes |
| Bosnian | Yes |
| Bulgarian | Yes |
| Catalan | Yes |
| Croatian | No |
| Chinese Simplified | Yes |
| Chinese Traditional | Yes |
| Danish | Yes |
| Dutch | Yes |
| Finnish | Yes |
| French | Yes |
| German | Yes |
| Georgian | No |
| Greek | Yes |
| Hebrew | Yes |
| Hungarian | Yes |
| Indonesian | Yes |
| Italian | Yes |
| Japanese | Yes |
| Korean | Yes |
| Latvian | Yes |
| Lithuanian | Yes |
| Malay | Yes |
| Norwegian | Yes |
| Polish | Yes |
| Portuguese | Yes |
| Portuguese Brazil | Yes |
| Romanian | Yes |
| Russian | Yes |
| Serbian | Yes |
| Slovak | No |
| Slovenian | Yes |
| Spanish | Yes |
| Swedish | Yes |
| Thai | Yes |
| Turkish | Yes |
| Vietnamese | Yes |
Please prevent from putting translations in PRs, I'd rather keep those for code. I try to check Transifex every other day, but if you'd like me to get a look at your translations quickly then hit me up on Gitter.