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

Use the official crowdin api client to upload and download translations. #453

Merged
merged 1 commit into from Aug 18, 2016

Conversation

aronasorman
Copy link
Collaborator

Summary

Changes:

  • Remove the unofficial python client dependency. Use the official crowdin client.
  • Add a new make command that uploads strings and then downloads the translations.

@aronasorman aronasorman added this to the MVP milestone Aug 18, 2016
@codecov-io
Copy link

codecov-io commented Aug 18, 2016

Current coverage is 73.99% (diff: 100%)

Merging #453 into master will not change coverage

@@             master       #453   diff @@
==========================================
  Files           116        116          
  Lines          3823       3823          
  Methods         143        143          
  Messages          0          0          
  Branches        444        444          
==========================================
  Hits           2829       2829          
  Misses          952        952          
  Partials         42         42          

Powered by Codecov. Last update 0b1499c...8e6d227

@@ -74,3 +74,8 @@ makemessages: assets

compilemessages:
python -m kolibri manage compilemessages -- -l en > /dev/null

syncmessages:

This comment was marked as spam.

This comment was marked as spam.

@aronasorman
Copy link
Collaborator Author

Comment addressed!

@rtibbles
Copy link
Member

I like it!

@rtibbles rtibbles merged commit 628b618 into learningequality:master Aug 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants