znuny / Znuny

Znuny/Znuny LTS is a fork of the ((OTRS)) Community Edition, one of the most flexible web-based ticketing systems used for Customer Service, Help Desk, IT Service Management.
https://www.znuny.org
GNU General Public License v3.0
335 stars 82 forks source link

Sector Nord AG: Fixed all missing german translations. #457

Closed paulfolkers closed 1 year ago

paulfolkers commented 1 year ago

Proposed change

In Znuny 7 there are many german translations missing, which will be available now.

1 - 🐞 bug 🐞

Additional information

Checklist

Units tests and code policy are not applicable to this.

dignin commented 1 year ago

Hi @paulfolkers , thanks for helping out here. If you have translation fixes in the future, please translate using the Weblate platform. https://translations.znuny.org/projects/znuny/znuny/de/ This will save you and us time in the long run. I've passed this on and @dennykorsukewitz will contact you if needed.

paulfolkers commented 1 year ago

Hi @paulfolkers , thanks for helping out here. If you have translation fixes in the future, please translate using the Weblate platform. https://translations.znuny.org/projects/znuny/znuny/de/ This will save you and us time in the long run. I've passed this on and @dennykorsukewitz will contact you if needed.

Hi Shawn, didn't know about that, thank you. Are you able to merge it or should I fill the missing translations at the Weblate platform?

dennykorsukewitz commented 1 year ago

Hi @paulfolkers ,

All files under Kernel/Language/* are automatically generated from the i18n/Znuny/Znuny.*.po via bin/otrs.Console.pl Dev::Tools::TranslationsUpdate. I can only merge the PR if you also edit the Znuny.de.po.

But as Shawn said, organizationally we want to separate the translations from the code here.

Anyway, thank you for your PR. In this one case you may decide whether to extend the PR or use Weblate. I am OK with both.

Regards :rocket:

paulfolkers commented 1 year ago

Hi @dennykorsukewitz, I placed all translation suggestions in Weblate. I did not create an account for that, we'd be happy if you could just mention Sector Nord AG in your changelog then. ;-)

I will close this PR for now, since this code merge is not necessary now.