Skip to content

feat(i18n): add Dutch nl-NL translations - #2118

Merged
ghostdevv merged 35 commits into
npmx-dev:mainfrom
WilcoSp:feat-i18n/adding-dutch
Apr 13, 2026
Merged

feat(i18n): add Dutch nl-NL translations#2118
ghostdevv merged 35 commits into
npmx-dev:mainfrom
WilcoSp:feat-i18n/adding-dutch

Conversation

@WilcoSp

@WilcoSpWilcoSp commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

📚 Description

Adding Dutch translations to npmx, I've also added nl as a language that has variants because of Flemish.

🧭 Context

This pr adds Dutch to the available languages, in case of "you" I've chosen for the formal variant ("u") instead of informal "je/jij"

terms usage (🇬🇧 = 🇳🇱)

  • package = pakket
  • dependency = dependency (stays same)
  • provenance = herkomst verificatie / herkomst gegevens
  • readme = readme (stays same)
  • deprecated = afgeraden ('not recommended' in Dutch)
  • like = vind leuk
  • unlike = vind niet leuk
  • contributor = bijdrager
  • facet = Aspect
  • playground = playground (stay same)
  • issue = issue (stays same)
  • patch = patch (stays same)
  • minor = sub (semver)
  • major = hoofd (semver)
  • operation = opdracht
  • command palette = actie menu (action menu, describes closer what it does and makes the placeholder translation possible, 'snelmenu' (quickmenu) was also an option but the placeholder couldn't be translated than)
  • command (only in command palette) = actie
  • home/homepage = start/startpagina
  • freshness = recentheid (for compare)
  • cookie = cookie (stays same, unless talking about an actual 🍪)

Translations that stay the same because these are already used by Dutch developers as is and because the direct translations don't fit the context of npmx & developing software.

🤖 AI & translator use

I did use Chatgpt, Gemini & Deepl to assist translating while going over each page & translation individually to get the best translations

progress

Below is the progress of translating

  • seo
  • footer
  • shortcuts
  • search
  • command_palette
  • nav
  • blog
  • settings
  • i18n
  • error
  • common
  • profile
  • package
  • connector
  • operations
  • org
  • user
  • claim
  • code
  • badges
  • filters
  • about
  • account_menu
  • auth
  • header
  • compare (except scatter_chart)
  • compare.scatter_chart (will be in a different pr)
  • pds
  • privacy_policy (will be in a different pr)
  • a11y
  • translation_status
  • vacations (will be in a different pr)
  • action_bar
  • logo_menu
  • brand
  • built_at + alt_logo + tagline + non_affiliation_disclaimer + trademark_disclaimer + alt_logo_kawaii

@vercel

vercelBot commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
npmx.devReadyReadyPreview, CommentApr 12, 2026 8:23pm
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devIgnoredIgnoredPreviewApr 12, 2026 8:23pm
npmx-lunariaIgnoredIgnoredApr 12, 2026 8:23pm

Request Review

@WilcoSpWilcoSp changed the title feat(i18n) adding Dutchfeat(i18n): adding DutchMar 17, 2026
@codecov

codecovBot commented Mar 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@WilcoSp

Copy link
Copy Markdown
ContributorAuthor

What's the progress on this? No pressure, just curious!

I'm still translating and getting further and further but my time is split between this pr, #1233, work and personal life.

I'll add an overview at the pr description to get a better indication how far the translations are

@WilcoSp

Copy link
Copy Markdown
ContributorAuthor

just temporarily disabled the "Allow edits and access to secrets by maintainers ", this is mostly due to autofix bot which when sync back can cause merge conflicts. When open for review it'll be enabled again.

@ghostdevv

Copy link
Copy Markdown
Member

What's the progress on this? No pressure, just curious!

I'm still translating and getting further and further but my time is split between this pr, #1233, work and personal life.

I'll add an overview at the pr description to get a better indication how far the translations are

No worries! If it helps we can always merge this partially translated, and then you (and/or others) can contribute more in follow up PRs - but it's up to you!

removed privacy policy & vacation (diff pr)
@coderabbitai

coderabbitaiBot commented Apr 12, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Added Dutch language support by registering a base nl locale and Netherlands variant in config/i18n.ts, and adding two new locale files (i18n/locales/nl.json and i18n/locales/nl-NL.json) containing Dutch translations and region-specific overrides.

Changes

Cohort / File(s)Summary
Localisation Configuration
config/i18n.ts
Registered nl as a base locale and added nlnl-NL mapping to countryLocaleVariants; enabled the active nl locale entry (previously commented).
Dutch Locale Files
i18n/locales/nl.json, i18n/locales/nl-NL.json
Added comprehensive Dutch translations (nl.json, ~1552 lines) and a small Netherlands-specific override file (nl-NL.json) with keys such as auth.modal.handle_placeholder.

Possibly related PRs

Suggested reviewers

  • ghostdevv
  • danielroe
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check nameStatusExplanation
Description check✅ PassedThe description is comprehensively related to the changeset, providing context about Dutch translations, term mappings, translation methodology, and detailed progress tracking across multiple UI sections.
Title check✅ PassedThe title 'feat(i18n): add Dutch nl-NL translations' accurately describes the main change in the pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@WilcoSp

Copy link
Copy Markdown
ContributorAuthor

@ghostdevv I've just translated the last things I think should be translated for this pr.
the only things I haven't translated are the privacy_policy, vacations & compare.scatter_chart.

  • for privacy policy the time will need to be taken to translate it so probably best for a different pr
  • vacation doesn't have any priority rn
  • compare.quadrant_chart or now compare.scatter_chart still very new and will probably still change a lot

@WilcoSp
WilcoSp marked this pull request as ready for review April 12, 2026 19:58
@WilcoSp

Copy link
Copy Markdown
ContributorAuthor

@coderabbitai review

@coderabbitai

Copy link
Copy Markdown
Contributor
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@i18n/locales/nl.json`:
- Line 365: The "not_found_message" translation contains a typo ("nit"); update
its value to correct Dutch ("Dit pakket kon niet gevonden worden.") and scan the
other high-visibility keys referenced in the comment (the other locale entries
at the given positions) for similar misspellings—replace "nit" with "niet" and
fix any other spelling/wording mistakes found so that all user-facing strings
read correctly in Dutch.
- Line 1133: The localization key "view_profile" in nl.json uses the Dutch token
"{naam}" which doesn't match the expected interpolation token "{name}" used by
callers; update the value for "view_profile" to use "{name}" so runtime
interpolation works (locate the "view_profile" entry and replace "{naam}" with
"{name}").
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 422cfe62-24cd-4e68-999c-baf65fc18616

📥 Commits

Reviewing files that changed from the base of the PR and between f70a11b and e99bd3c.

📒 Files selected for processing (3)
  • config/i18n.ts
  • i18n/locales/nl-NL.json
  • i18n/locales/nl.json

Comment threadi18n/locales/nl.json Outdated
Comment threadi18n/locales/nl.json Outdated
"$schema": "../schema.json",
"auth": {
"modal": {
"handle_placeholder": "oranje.npmx.dev"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

what's this? 👀

@WilcoSpWilcoSpApr 13, 2026

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Alice isn't a common name here in the Netherlands so I changed it to the national color 🟧 instead to be more recognisable

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ah, awesome!

@ghostdevvghostdevv changed the title feat(i18n): adding Dutchfeat(i18n): add Dutch nl-NL translationsApr 13, 2026
@ghostdevv
ghostdevv added this pull request to the merge queueApr 13, 2026
Merged via the queue into npmx-dev:main with commit 7f1e259Apr 13, 2026
22 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Apr 13, 2026
ayo-run pushed a commit to ayo-run/npmx.dev that referenced this pull request Aug 5, 2026
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for freeto 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

@WilcoSp@ghostdevv