Build new translationtool.phar

[nickvergessen]

Fix #331

@mgallien do you know how/where to deploy this? Is merging to master enough? Or does it need manual pulling somewhere?

[mgallien]

going to try to remember what is needed
at least a new version of the docker image should be generated

[mgallien]

@nickvergessen from my understanding our scripts do not specify a tag when running the docker image to handle server translations (see https://github.com/nextcloud/docker-ci/blob/master/translations/transifexSync.php#L69)

[nickvergessen]

Okay, so we merge and tag and it should be fine?

[mgallien]

Okay, so we merge and tag and it should be fine?

from my understanding yes
after you have pushed a tag, remember that you need to trigger the image build through a github action

[nickvergessen]

Hopefully I did everything correct?
https://github.com/nextcloud/docker-ci/actions/runs/1654392708
I had no clue what to put in the fields.

[rakekniven]

Will this improvement work for the whole Nextcloud org at GitHub or server repo only?

[nickvergessen]

Should work for apps and seever since Monday

[rakekniven]

There was one vue commit yesterday at polls and it has no comment.
Maybe it was bad timing.

[nickvergessen]

Spreed was synced and doesn't have it either. Can you create me a screenshot where the translator hints show normally in transifex?

[rakekniven]

This is an example of an iOS string.

Link:
https://www.transifex.com/nextcloud/nextcloud/translate/#de_DE/$/155338305?q=developer_notes%3Abutton