Skip to content

Updates to German translation #529

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 9, 2022
Merged

Updates to German translation #529

merged 2 commits into from
Mar 9, 2022

Conversation

rustermi
Copy link
Contributor

@rustermi rustermi commented Mar 9, 2022

Since we now have the newest version (1.0.67) running in production, I found a few things this PR fixes now:

  1. Renamed messages_de_DE.properties

    The translation is understandable for all German speaking people. However, the previous file name lead to these translations only being used for Germany (de_DE — German on German territory). A JVM running with Austrian German (de_AT) or simply German without any territory descriptor (de) resulted in the English default messages being used.

  2. Update German translations

    • 'false' translation was missing
    • Wording has been improved for cases with zero, one or many elements
    • Capitalisation of words after a colon has been unified

Sorry for the inconvenience and thank you for your work.

rustermi added 2 commits March 9, 2022 07:42
The translation is understandable for all German speaking people.
However, the previous file name resulted in these translations only
being used for Germany (de_DE). A JVM running with Austrian German
(de_AT) or simply German (de) would have resulted in the English
default messages being returned.
- 'false' translation was missing
- Wording has been improved for cases with zero, one or many elements
- Capitalisation of words after a colon has been unified
@stevehu stevehu merged commit af6bba1 into networknt:master Mar 9, 2022
@stevehu
Copy link
Contributor

stevehu commented Mar 10, 2022

@rustermi Thanks a lot for your help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants