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

Jump to bottom

Enable Internationalization #299

Closed
12 tasks done
lidel opened this issue Oct 22, 2017 · 0 comments · Fixed by #302
Closed
12 tasks done

Enable Internationalization #299

lidel opened this issue Oct 22, 2017 · 0 comments · Fixed by #302
Assignees
@lidel
Labels
exp/novice Someone with a little familiarity can pick up kind/maintenance Work required to avoid breaking changes or harm to project's status quo P3 Low: Not priority right now

Comments

@lidel
Copy link
Member

lidel commented Oct 22, 2017 •
edited

The WebExtensions API has a rather handy module available for internationalizing extensions — i18n.
Refactoring will be a chore, but needs to be done.

Remaining work:

  • Create new branch for localization-related work
  • Remove unused locale keys
  • Rename legacy keys used now in multiple contexts (eg. copy link)
  • Create new keys for static strings (eg. entire Preferences screen)
  • Make it easy to crowd source translations from non-technical people that are not comfortable with Github interface. DONE: https://crowdin.com/project/ipfs-companion
    • Request free project via https://crowdin.com/page/open-source-project-setup-request
    • Get approved by Crowdin:

      Your Open Source project has been APPROVED at Crowdin! We're looking forward to see your applications localized. 🚀

    • Set up Crowdin integration for Github project
      wont happen -- they wanted full read/write rights to repo, I'll sync translations manually instead
  • Update README and CONTRIBUTING guide to include info how to help with translation
  • Ask community for help with translating :) https://discuss.ipfs.io/t/maybe-we-need-more-languages/1332/
  • Release new version with at least one translation different than English
    • Finish pl_PL
    • Sync translation from Crowdin to Github

References
@lidel lidel added exp/novice Someone with a little familiarity can pick up kind/maintenance Work required to avoid breaking changes or harm to project's status quo P3 Low: Not priority right now labels Oct 22, 2017
@lidel lidel self-assigned this Oct 22, 2017
@lidel lidel changed the title Internationalization Enable Internationalization Oct 28, 2017
@lidel lidel mentioned this issue Oct 28, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lidel lidel

Labels
exp/novice Someone with a little familiarity can pick up kind/maintenance Work required to avoid breaking changes or harm to project's status quo P3 Low: Not priority right now
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add first batch of locales ipfs/ipfs-companion
1 participant
@lidel