Skip to content

Added dutch translations to client#966

Open
stijnie2210 wants to merge 3 commits into
rybbit-io:masterfrom
stijnie2210:dutch-translations
Open

Added dutch translations to client#966
stijnie2210 wants to merge 3 commits into
rybbit-io:masterfrom
stijnie2210:dutch-translations

Conversation

@stijnie2210
Copy link
Copy Markdown
Contributor

@stijnie2210 stijnie2210 commented Apr 6, 2026

Added support for Dutch in the Rybbit client

Summary by CodeRabbit

  • New Features
    • Added Dutch (Nederlands) language support — users can now select Dutch in the language switcher. Locale detection and fallback recognize and load Dutch. The app includes comprehensive Dutch localization (~1,200+ strings) covering UI text, status/error messages, analytics and reporting flows, exports, session playback, onboarding/billing, tracking prompts, and parameterized messages.

Review Change Stack

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 6, 2026

@stijnie2210 is attempting to deploy a commit to the goldflag's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7208278d-a477-4e5e-b68d-ea621bf74fad

📥 Commits

Reviewing files that changed from the base of the PR and between 4fcec36 and b3fa010.

📒 Files selected for processing (1)
  • docs/src/app/[locale]/layout.tsx
✅ Files skipped from review due to trivial changes (1)
  • docs/src/app/[locale]/layout.tsx

📝 Walkthrough

Walkthrough

Added Dutch locale support: new client/messages/nl.json (1,217 strings), nl added to the language switcher, and nl registered in i18n request and Next.js intl configuration.

Changes

Dutch Localization

Layer / File(s) Summary
Dutch messages resource
client/messages/nl.json
Added new Dutch (nl) message resource containing 1,217 localized key/value strings covering UI text, errors, status messages, reporting, billing, tracking, and account flows.
Locale registration and UI
client/src/components/LanguageSwitcher.tsx, client/src/i18n/request.ts, client/next.config.ts, docs/src/app/[locale]/layout.tsx
Registered nl in supported locales and Next.js intl config, added Nederlands/NL entry to the language switcher, and updated docs i18n translations with displayName: 'Nederlands'.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • rybbit-io/rybbit#905: Adds a locale JSON, updates the language switcher, and registers the locale in i18n config (same pattern as this PR).
  • rybbit-io/rybbit#958: Adds a locale end-to-end and updates the same i18n integration points as this PR.

Poem

🐰 I hopped in with a cheerful cheer,
Nederlands landed, bright and near,
Keys and strings in tulip light,
Messages dancing through the night,
A tiny rabbit says: heerlijk — hooray! 🇳🇱

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Added dutch translations to client' directly and accurately describes the main change: adding Dutch language support to the client application with message translations and locale configuration.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this 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

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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.

Copy link
Copy Markdown

@GijsKruize GijsKruize left a comment

Choose a reason for hiding this comment

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

All dutch translations look good

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