-
Notifications
You must be signed in to change notification settings - Fork 490
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
docs: i18n #783
docs: i18n #783
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've found 2 places that sound funny, if you fix them LGTM.
README.md
Outdated
|
||
The translations are stored on [./public/locales](./public/locales) and the English version is the source of truth. | ||
|
||
We use Transifex to help us translate WebUI to another languages. If you're interesting in contributing, go to [our page on Transifex](https://www.transifex.com/ipfs/ipfs-webui/translate/), create an account, pick up a language and start translating. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
interesting
→ interested
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😆
README.md
Outdated
#### To Start Translating | ||
|
||
1. [Create Transifex account](https://www.transifex.com/signup/?join_project=ipfs-webui) | ||
2. Go to https://www.transifex.com/ipfs/ipfs-webui/translate/, pick up the language, and start translating |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pick up the language
→ pick a language
Done and merged @lidel |
@lidel I stole some of your words on #772!
Could you take a look and add/update anything if you feel to?