jamulus icon indicating copy to clipboard operation
jamulus copied to clipboard

Translations update from Hosted Weblate

Open weblate opened this issue 1 year ago • 12 comments

Translations update from Hosted Weblate for Jamulus/Jamulus app.

It also includes following components:

Current translation status:

Weblate translation status

weblate avatar Sep 15 '22 15:09 weblate

@ann0see I do that so I know what I am unsure about. Mostly I do it to keep track of what source strings need fixing.

comradekingu avatar Sep 17 '22 12:09 comradekingu

@jujudusud great. It's just that it would be good to have one approver per language before it gets merged.

ann0see avatar Sep 17 '22 12:09 ann0see

@ann0see If you want to review everything I have done, run it through auto-translation. The quality of the source strings isn't good enough to warrant complaining about anyone else. Removing the "needs editing" flags makes it much harder for me to change that.

comradekingu avatar Sep 17 '22 14:09 comradekingu

@comradekingu What's so terribly wrong with the source strings? Could you give some examples? Sure some strings could possibly be made clearer, but you do seem to sound quite dismissive of them as a whole, as if they're so bad a proper translation isn't possible. You're the first translator to raise this as an obstacle for translating.

ignotus666 avatar Sep 17 '22 14:09 ignotus666

If you want to review everything I have done, run it through auto-translation.

No need for that. Usually we let the translators decide and ask for other potential reviewers if they exist to spot spelling mistakes etc. If you're not yet ok with some translations that's totally fine (the translator is mostly in charge of his/her own translation anyway). As soon as we reach release candidate stage, the existing strings should be "releasable".

ann0see avatar Sep 17 '22 20:09 ann0see

What's so terribly wrong with the source strings?

I think there are some unclear strings - at least I find some once in a while. Best would be to raise an issue and list them with a suggestion + explanation what needs fixing

ann0see avatar Sep 17 '22 20:09 ann0see

Review checklist:

  • [ ] Translator listed in the src/util.cpp optionally add link to PR or code (todo: Add kingu and invite to matrix/discord)
  • [ ] Punctuation and spacing consistent
  • [x] Signal words consistent ("ASIO", "Buffer")
  • [ ] App translations: No untranslated strings (grep unfinished -5 src/translation/translation_$TRANSLATION*.ts) (not to be checked in this stage)
  • [x] App translations: Only a single .ts file checked in (.qm in addition is also OK)

ann0see avatar Sep 18 '22 20:09 ann0see

@ignotus666 The source strings aren't too far off, but I'd rather do something about it. This way it becomes far easier to translate Jamulus for yet others. I don't have a lot of time, but I'll have a look and see what files need changing to update the source strings correctly.

comradekingu avatar Sep 19 '22 15:09 comradekingu

We also need to add the information for the translators in the translation.md file and in each issue created.

jujudusud avatar Sep 21 '22 13:09 jujudusud

Plus what ever process documents adding a new translator needs to be reviewed -- ideally so we've got an issue template we can use.

pljones avatar Sep 21 '22 15:09 pljones

Fixes #2867

I approuved all the strings. Waiting for Weblate to push the commit.

jujudusud avatar Sep 22 '22 21:09 jujudusud

Fixes #2870

henkdegroot avatar Sep 24 '22 09:09 henkdegroot

https://hosted.weblate.org/translate/jamulus/jamulus-app/en/?q=has:comment#comments Can that lot get closed?

pljones avatar Sep 28 '22 21:09 pljones

@pljones those messages on weblate are gone now.

ignotus666 avatar Sep 28 '22 21:09 ignotus666

@hoffie yes, @comradekingu is missing in the utils.h file, as far as I know.

ann0see avatar Oct 10 '22 19:10 ann0see