Skip to content

Dutch translation file - #54

Merged
FarmLox merged 1 commit into
no-faff:mainfrom
RijckAlex:main
Jul 24, 2026
Merged

Dutch translation file#54
FarmLox merged 1 commit into
no-faff:mainfrom
RijckAlex:main

Conversation

@RijckAlex

Copy link
Copy Markdown
Contributor

Want to add the Dutch translation file

New translation file nl.md

What this changes

Full translation

Why

Only to add the Dutch translation

How it's been tested

  • dotnet build succeeds
  • dotnet test passes
  • Ran the app locally and verified the change

Notes for the reviewer

Just add it and the Dutch-speaking population will be satisfied.

Thanks!

Want to add the Dutch translation file
@FarmLox

FarmLox commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Wow. Nice! Thanks very much.

I'm onto it now. You'll get a README too. I'll let you know in here when it's all ready. Have a look at the app, obviously, when it's ready but the README too if you get a chance - but it's a lot of words, I know. I'd be delighted if you can identify any improvements I can make.

Much appreciated. I'll be back again soon.

@FarmLox
FarmLox merged commit 094067c into no-faff:main Jul 24, 2026
5 checks passed
FarmLox added a commit that referenced this pull request Jul 25, 2026
RijckAlex contributed a complete Dutch translation unprompted: every window,
dialog, tooltip, button and screen-reader label, plus the command-line tool.
It arrived as an edit to the generated review table rather than to a source
the app reads, so the content is carried over into the generator that feeds
Strings.nl.resx instead of being merged as it stood.

The translation itself needed a pass before it could ship. It mixed je and u
throughout, so the register is settled on one; and roughly one row in ten
carried a real error, among them an untranslated French cell, doubled {0}
placeholders and several mistranslations. Dutch count agreement is handled
rather than calqued from the English.

Dutch is last in the language order, after Ukrainian, on the developer-audience
rule the other fifteen already follow. The globe menu is unaffected, being
alphabetical by endonym.

check-installer-languages.mjs hard-fails a language that skips the installer,
so the installer carries Dutch wizard text with it; check-accelerators,
check-still-english and gen-translation-table all take the sixteenth code.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
FarmLox added a commit that referenced this pull request Jul 25, 2026
His file and ours both add docs/translations/nl.md, so the merge stops on it.
Resolved to ours, which the generator produces from Strings.nl.resx and which
check-still-english and gen-translation-table hold in step with what the app
actually says.

His could not be kept as it stood. It is the generated review table, not a
source the app reads, and it was built by copying the French one, so its header
still named French and pointed at Strings.fr.resx. His wording is what feeds
the generator; this merge keeps his commit so the contribution is his in the
history.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@FarmLox

FarmLox commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

v2.3.0 is released, @RijckAlex

Thanks ever so much again!

If you do get a chance to skim through the Dutch README and see especially if there are any glaring mistakes, that would be great.

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.

2 participants