@sgiehl opened this Pull Request on April 14th 2015 Member

As part of #7567 we need to switch our console commands to use Transifex.com API instead of oTrance.
The new implementation is a lot slower than before as we currently have a resource per language file (plugin). So we have 57 resources and 66 languages. The current implementation sends around 3700 api requests to transifex to update each resource and language as I didn't find a way to load "bulk data" eg. all translations over all resources and/or languages.
I've asked the tech support from transifex if there is any possibility to do that.
Let's see what they answer, otherwise the update will have a duration of about 10 minutes. (Which would be ok, I guess, as we are not doing it that often)

@mnapoli commented on April 14th 2015 Contributor

Is it possible to do the "first translation update" in a separate pull request or would it be too much of a hassle? That would allow to review the diff (when the PR is ready for review), right now there are 600 files changed and the diff is too huge to load.

@sgiehl commented on April 15th 2015 Member

Good point. I've removed the commit.

@sgiehl commented on April 20th 2015 Member

Good for review. As soon as it was merged, I'll create a first automatic translation update pull request using the new transifex api

@mnapoli commented on April 20th 2015 Contributor

Looks good to me, I've commented minor details. Just out of curiosity why are "fetch translations" and "update translations" separate operations?

Could you also rebase on master (or merge master), so that the unit tests are fixed?

@mattab commented on April 21st 2015 Owner

Nice progress @sgiehl

I'm curious: did you hear from Transifex support about possible speed improvements to API imports?

@sgiehl commented on April 21st 2015 Member

@mnapoli I don't know the reason why I have splitted that in the first place. Maybe as it was easier to build that part by part.
I've changed some of the stuff you commented and rebased on master. I'll merge it now and create a new pull request for translation updates using the new stuff :)

This Pull Request was closed on April 21st 2015
Powered by GitHub Issue Mirror