Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

Streamline i18n tooling and added French Canadian, German as well as Japanese #158

Merged
merged 3 commits into from
Aug 1, 2019

Conversation

OmarIthawi
Copy link
Contributor

@OmarIthawi OmarIthawi commented Jul 24, 2019

I've been adding standard-ish i18n tooling for various popular Open edX XBlocks e.g. poll xblock and Google Drive. The end-goal is to make the i18n workflow as seamless as possible.

The UBCPI XBlock already has most of the required tooling (thank you), so I'll be adding only few additional commands.

New Additions

  • Added two Makefile commands: pull_translations, push_translations also amended test to ensure proper file translations.
  • Added the following languages:
    • French Canadian
    • Japanese
    • German

Bonus

I also have plans to make a i18n bot that updates the translations to save on engineering hours.

@coveralls
Copy link

coveralls commented Jul 24, 2019

Coverage Status

Coverage increased (+0.04%) to 96.707% when pulling f984e3c on appsembler:omar/i18n-streamline into 372497d on ubc:master.

@OmarIthawi
Copy link
Contributor Author

Hi @xcompass @richardtape, Could you please take a look at this pull request? Thank you!

@OmarIthawi
Copy link
Contributor Author

@AhmedAljazzar and @melvinsoft, could one of you please also take a quick look here?

Copy link

@iamjazzar iamjazzar left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Contributor

@kitsook kitsook left a comment

Choose a reason for hiding this comment

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

Thanks for submitting the PR. Some questions on the removed zh_CN translations.

ubcpi/static/js/src/translations.js Outdated Show resolved Hide resolved
@kitsook kitsook merged commit 998a52f into ubc:master Aug 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants