[#796] - Crowdin #2
- Fixed in Code Base
- 19 Jun 2016
- Medium
- Build: master
- # 796
- Diff
- joomla:crowdin-kuku
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success continuous-integration/travis-ci/push The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #787
This is an alternative to #795 with less code added
Summary of Changes
- Methods have been added to allow file exchange with https://crowdin.com
- The library elkuku/crowdin-api has been added to the composer file.
- Properties for crowdin credentials have been added to /etc/config.dist.json
Testing Instructions
Upload new language templates
jtracker update languagefiles --provider=crowdin --create
Upload already existing translated files to crowdin
jtracker update languagefiles --provider=crowdin --translations
This should only run once.
Download translated files
jtracker get languagefiles --provider=crowdin
Hint: I very personally would prefer this one over #795
Note that methods have been consolidated in a B/C braking way now...
Well, glad we never tagged a "stable" version
I'd say sync this with master to make sure the service stuff didn't screw anything up and this should be good to go.
Synced.
Seems like it works (did a test fetch from Transifex with this branch and it pulled files correctly).
| Status | New | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-06-19 16:37:00 |
| Closed_By | ⇒ | mbabker |
did a test fetch from Transifex
Nice test...
I went with what I had available, which made sure all the application relative code worked right. I hope your package is tested and works
I hope your package is tested
Well it has 100% unit test coverage... but since the only "real" tests are in the real world, the jissues project would have the honour to report the first issue then :)
Actually I just hit the button twice and didn't realize it squashed it until too late.
I turned off the squash merge option. I didn't realize that GitHub was defaulting to this now. Guess I should RTFI once in a while.
Al good - stuff happens
Note that methods have been consolidated in a B/C braking way now...
A
providerparameter is now required to execute the commands. E.g.Test instructions have been updated.