Dutch out of Beta: the native speaker's final review, from public ihasmail #23

Merged

jcoffey-dev merged 3 commits from i18n/dutch-native-review into main

2026-09-27 23:01:43 +00:00

jcoffey-dev commented

2026-09-27 22:58:05 +00:00

Owner

Takes public ihasmail #14: Michael's final review of both Dutch catalogs, Dutch out of Beta, and his credit.

Cherry-picked rather than merged: public ihasmail's history was rewritten after the last merge-down, so a merge replays three weeks the fork already has.

One conflict in nl.ts: the reviewed wording is taken, on the fork's key that says "the mail server" rather than the upstream name.
FEATURES, KNOWN-ISSUES and ROADMAP stay deleted in the fork; the README keeps the fork's wording.
Fork-only Dutch strings (the delegated-accounts strings among them) are not reviewed; the catalog header already says strings added after the review haven't been read.

1431 tests pass.

Takes public ihasmail #14: Michael's final review of both Dutch catalogs, Dutch out of Beta, and his credit. Cherry-picked rather than merged: public ihasmail's history was rewritten after the last merge-down, so a merge replays three weeks the fork already has. - One conflict in `nl.ts`: the reviewed wording is taken, on the fork's key that says "the mail server" rather than the upstream name. - FEATURES, KNOWN-ISSUES and ROADMAP stay deleted in the fork; the README keeps the fork's wording. - Fork-only Dutch strings (the delegated-accounts strings among them) are not reviewed; the catalog header already says strings added after the review haven't been read. 1431 tests pass.

jcoffey-dev added 3 commits 2026-09-27 22:58:05 +00:00

Take Dutch out of Beta with the native speaker's final review ab759143ab

Michael (mbjboon-netizen) sent the final corrections for nl.ts and the Dutch
permission labels and signed the language off, so Nederlands no longer
carries the Beta flag in the picker.

The main catalog changes 52 values, mostly "regels" -> "filterregels" and
"post" -> "e-mail(s)". The permission headings move from compound nouns
("Accountbeheer") to verb phrases ("Accounts beheren"), and the reviewer's
note on that is kept in the file. No keys were added or removed, and every
placeholder is intact.

One entry is kept as it was: "It {damage}, ..." stays "Het {damage}, ...".
{damage} is filled with a verb phrase ("stops in the middle of a line"), so
the added "is" would have doubled the verb.

README, FEATURES, ROADMAP and KNOWN-ISSUES now say Dutch has been reviewed
and the other eight have not.

(cherry picked from commit e30fd73d7dbb1463859efbc4048f680d21d64c56)

Credit Michael (mbjboon82) for the Dutch review f95072ab92

Name the reviewer in both Dutch catalogs, FEATURES and ROADMAP, under both of his handles, and record that his wording stands.

(cherry picked from commit aaa86e96d66e2431a8c98467712a977d4035bf76)

Link Michael's profile from the Dutch credits

ci / version (pull_request) Skipped

Details

ci / node (pull_request) Successful in 2m40s

Details

ci / publish (pull_request) Skipped

Details

ci / announce (pull_request) Skipped

Details

ci / docker-build (pull_request) Successful in 30s

Details

2c4d6cdfae

(cherry picked from commit 9840a537834082bfd9a028a170ce2d6ac60ddf35)

jcoffey-dev merged commit 2361caf2c7 into main

2026-09-27 23:01:43 +00:00

jcoffey-dev deleted branch i18n/dutch-native-review

2026-09-27 23:01:43 +00:00

jcoffey-dev referenced this pull request

2026-09-27 23:28:13 +00:00

Seven app fixes from public ihasmail #25

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: inbuxa/inbuxa-webmail#23