Skip to content

Update nl.json - completion and corrections to Dutch#5419

Open
Stephan-P wants to merge 5 commits intoNginxProxyManager:developfrom
Stephan-P:patch-1
Open

Update nl.json - completion and corrections to Dutch#5419
Stephan-P wants to merge 5 commits intoNginxProxyManager:developfrom
Stephan-P:patch-1

Conversation

@Stephan-P
Copy link
Copy Markdown

@Stephan-P Stephan-P commented Mar 21, 2026 •
edited
Loading

Adding Dutch language translation for latest additions
Capitalisation
Infinitive verb form

@Stephan-P
Update nl.json - completion and corrections
40f39a7 
Adding Ducth language translation for latest additions
Capitalisation
Infinitive verb form
@Stephan-P
Copy link
Copy Markdown
Author

Stephan-P commented Mar 21, 2026 •
edited
Loading

Docker Image for build 1 is available on DockerHub:

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

This is my first contribution to NPM and I've never experienced something like this with other projects where I volunteer Dutch translations.

  • Am I supposed to verify my updated translations by replacing the container reference in my compose file?
  • How much risk would be involved for my existing NPM setup?

@Stephan-P
Update nl.json
82a326e 
Consistency in Dutch translation of 'enable' and 'disable'
TheMazeIsAmazing
TheMazeIsAmazing suggested changes Apr 13, 2026
View reviewed changes
Copy link
Copy Markdown

@TheMazeIsAmazing TheMazeIsAmazing left a comment

Choose a reason for hiding this comment

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

Hi!
Basically all the new / rewritten lines are great, here are some suggestions to improve them 😉.
Thanks for your effort!

Comment thread frontend/src/locale/src/nl.json Outdated
Comment thread frontend/src/locale/src/nl.json Outdated
Comment thread frontend/src/locale/src/nl.json Outdated
Comment thread frontend/src/locale/src/nl.json Outdated
Comment thread frontend/src/locale/src/nl.json Outdated
@Stephan-P
Update nl.json
871787a 
fixes for CRs by @TheMazeIsAmazing
Thanks. Well spotted!
@Stephan-P
Update nl.json
a03c479 
More updates after running the PR-version on live server.
Applied consistency with https://nl.wikipedia.org/wiki/Lijst_van_HTTP-statuscodes
@Stephan-P
Copy link
Copy Markdown
Author

Stephan-P commented Apr 13, 2026

Managed test nginxproxymanager/nginx-proxy-manager-dev:pr-5419 and found some more stuff for improvement, e.g. HTTP status codes

@Stephan-P
Update nl.json
218e32b 
Hyphen inserted for "404-Host"
@nginxproxymanagerci
Copy link
Copy Markdown

nginxproxymanagerci bot commented Apr 13, 2026

Docker Image for build 5 is available on DockerHub:

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

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!

@Stephan-P
Copy link
Copy Markdown
Author

Stephan-P commented Apr 13, 2026

@TheMazeIsAmazing Unless there'll be source changes, then I guess that the Dutch translation is completed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@TheMazeIsAmazing TheMazeIsAmazing TheMazeIsAmazing requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Stephan-P @TheMazeIsAmazing