Skip to content

Conversation

@lastsamurai26
Copy link

@jc21
I finally managed to install the Dev environment.

Here is the German translation.
I tested it and adjusted a few things, but it's difficult to do this accurately in German.

In German, we say

"Bist du dir sicher das du diese Zugriffsliste löschen möchtest?"

in english is it always this - Are you sure you want to delete this Access List?

"Bist du dir sicher das du diesen Proxy Host löschen möchtest?"

in english aswell this - Are you sure you want to delete this Proxy Host?

so i did this "diese(n)" i hope this is ok

@jc21
Copy link
Member

jc21 commented Nov 3, 2025

I'll leave this here for a few days and see if any other German speakers have thoughts

@Skeptic-systems
Copy link

Skeptic-systems commented Nov 4, 2025

The translation looks good as far as I can tell. @lastsamurai26 Just a tip, you should work on your commit messages :) -> https://www.conventionalcommits.org/

@lastsamurai26
Copy link
Author

The translation looks good as far as I can tell. @lastsamurai26 Just a tip, you should work on your commit messages :) -> https://www.conventionalcommits.org/

Thank you, I'll make a note of that. :-)
Since I usually only do this for myself,
I suppose I'll have to do it for the general public.

Copy link

@fhennig42 fhennig42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was waiting for the CI to build my PR, so I took a look into this one while waiting for the build.
Even though I'm not a nginx contributor/maintainer (yet) I would suggest taking a look into my suggestions before merging/approving the PR.

"defaultMessage": "Maximum ist {max}"
},
"error.min-character-length": {
"defaultMessage": "Die mainimale Länge beträgt {min} Zeichen{min, plural, one {} other {s}}"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same typo from above, is this file autogenerated?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes but not from me
the dev enviroment from nginx re generate the files after each chang

Comment on lines +387 to +394
"defaultMessage": "We are sorry but the page you are looking for was not found"
},
"notfound.title": {
"defaultMessage": "Oops… You just found an error page"
},
"notification.error": {
"defaultMessage": "Error"
},

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no translations here?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

=( Yes, unfortunately, because I couldn't find a suitable translation. And this one sounds okay to me as a German speaker.

@lastsamurai26
Copy link
Author

Thank you for checking, but it is strange why the parts are in there, as I did not use the tool. To be honest, I hate AI systems or tools. However, I suspect that the dev environment where I translated everything had something installed that caused this. I will check this later at work, as this was a dev environment from work.

I didn't translate the last three points in your suggestion because, strangely enough, I couldn't find them in nginx. If you have a suggestion, I'd be happy to hear it.

I'm not sure whether your suggestions will be accepted automatically. :( I do work with git, but mostly on my own.

fix: typo 
fix: translate Location with Pfad
fix: typo mainimale should be minimale
fix: typo dark and light mode
fix: translation "Umgangssprachlich"
fix: typos
Fix: German translations
@nginxproxymanagerci
Copy link

Docker Image for build 4 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-4824

Note

Ensure you backup your NPM instance before testing this image! Especially if there are database changes.
This is a different docker image namespace than the official image.

Warning

Changes and additions to DNS Providers require verification by at least 2 members of the community!

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.

4 participants