search

owncloud / music

:notes: Music app for ownCloud
GNU Affero General Public License v3.0
565 stars 197 forks source link

Missing translations - Polish language #712

Open TitanSpirit opened 5 years ago

TitanSpirit commented 5 years ago

I would like to offer translation from English to Polish, how can I help?

paulijar commented 5 years ago

Hello, and thanks for the offer. Translations for this app are created in Transifex within the owncloud-org project. In principle, you should be able to contribute translations if you navigate to https://www.transifex.com/signin/?next=/owncloud-org/, log in (you can use your github account), and join the Polish translation team.

However, we are currently having an integration issue with Transifex: only a small portion of all the localizable strings of the Music app show up in Transifex. All the strings currently available on Transifex already seem to have a Polish translation. The mentioned integration issue is tracked at https://central.owncloud.org/t/owncloud-music-app-translations/14881.

paulijar commented 3 years ago

As the Transifex problem has not been moving anywhere, I guess we should forget it for now. Instead, see the following post for an offline process to contribute translations: https://github.com/owncloud/music/issues/671#issuecomment-782746463

simonspa commented 3 years ago

Hi @paulijar

I am also interested in contributing translation strings, but reading up on the linked topic it seems like ownCloud is not interested in this at all. I understand that you were already in contact with @jancborchardt concerning a move to Nextcloud - is this something you are following up on?

In the meantime I'll follow the manual approach suggested in #671

Cheers, Simon

p-bo commented 1 year ago

@paulijar Regarding migration to Nextcloud Transifex - just open issue on https://github.com/nextcloud/docker-ci/issues/new/choose (choose Request translations). And you will get checklist as for example this: https://github.com/nextcloud/docker-ci/issues/426 and you can ask rakekniven if something is unclear to you.

p-bo commented 1 year ago

For motivation - this developer was very impressed how much work can be done by Nextcloud translators community :-)

paulijar commented 1 year ago

@p-bo Thanks for the hint. This looks interesting; I need to look into it closer once I have time.

paulijar commented 1 year ago

@rakekniven If I would do as suggested above and registered this app to be translated using Nextcloud Transifex, then would that necessarily mean that the bot would automatically use the Nextcloud translation tool to scrape the translatable strings and update the .pot file? Because that would almost certainly be the wrong way for this project and it would find only a tiny fraction of all the translatable strings. That's basically the reason why we had to disable the ownCloud Transifex integration. Most of our translations don't use functions t() or n() but are implemented using angular-gettext.

If, however, it would be possible for me to update the .pot file manually (using suitable scripts, of course), and the bot would update only the .po files, then this could be a solution. Alternatively, it could work if there was a way to pass a custom command for the bot to extract the translatable strings.

rakekniven commented 1 year ago

@nickvergessen can you answer this?

rakekniven commented 1 year ago

then would that necessarily mean that the bot would automatically use the Nextcloud translation tool to scrape the translatable strings and update the .pot file?

Yes, the mechanism is the same and therefor not usable for your use case, sorry.

paulijar commented 1 year ago

Is there any chances that one day in the future it could be more configurable? I imagine that the bot works in phases something like

  1. Pull the latest changes from github
  2. Scrape the code to regenerate .pot
  3. Upload the .pot to Transifex
  4. Download all the translated .po files from Transifex
  5. Push all the updated files to github

Now, if we could opt out from the phase 2 and keep the rest, then everything would be wonderful.

rakekniven commented 1 year ago

Actually I am not very deep in the code for this but AFAIK it works like you describe (step 4 does more as there are json files as well).

Is there any chances that one day in the future it could be more configurable?

Feel free to dive in into https://github.com/nextcloud/docker-ci/tree/master/translations to improve/adapt scripts