Skip to content

fix(localizations): use future tense for nl-NL and nl-BE - #9523

Closed
jorisvdz wants to merge 2 commits into
clerk:mainfrom
jorisvdz:main
Closed

jorisvdz wants to merge 2 commits into
clerk:mainfrom
jorisvdz:main

Conversation

@jorisvdz

Copy link
Copy Markdown
Contributor

The email and phone verification copy implied the code had already been sent. These screens send the code after the user continues, so the Dutch and Belgian strings now say it will be sent.

@changeset-bot

changeset-bot Bot commented Aug 21, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 44ffda9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@clerk/localizations Patch
@clerk/ui Patch
@clerk/chrome-extension Patch
@clerk/swingset Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

@jorisvdz is attempting to deploy a commit to the Clerk Production Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 1430f997-6524-47b3-bdd6-a5508aa99128

📥 Commits

Reviewing files that changed from the base of the PR and between 9abe560 and 44ffda9.

📒 Files selected for processing (1)
  • .changeset/open-teeth-hunt.md
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • clerk/clerk_go (manual)
  • clerk/dashboard (manual)
  • clerk/accounts (manual)
  • clerk/backoffice (manual)
  • clerk/clerk (manual)
  • clerk/clerk-docs (manual)
  • clerk/cloudflare-workers (manual)

Included review availability: Your plan provides up to 4 included reviews per hour; 2 remain after this review.


📝 Walkthrough

Walkthrough

Updated the email and phone verification hints in the Dutch and Dutch-Belgian localization files. The hints now state that verification messages will be sent. Added a patch changeset for @clerk/localizations.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: wobsoriano

Merge Risk: ⚪ Minimal · up to 44ffd

This PR corrects Dutch and Belgian Dutch verification hints to describe codes being sent after continuation. The change is limited to user-facing wording and is ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the localization fix and specifies the affected locales, nl-NL and nl-BE.
Description check ✅ Passed The description accurately explains the tense correction for email and phone verification messages and matches the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2…
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.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch

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

@wobsoriano wobsoriano left a comment

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.

thanks!

@pkg-pr-new

pkg-pr-new Bot commented Sep 9, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@9523

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@9523

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@9523

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@9523

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@9523

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@9523

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@9523

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@9523

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@9523

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@9523

@clerk/express

npm i https://pkg.pr.new/@clerk/express@9523

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@9523

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@9523

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@9523

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@9523

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@9523

@clerk/react

npm i https://pkg.pr.new/@clerk/react@9523

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@9523

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@9523

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@9523

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@9523

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@9523

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@9523

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@9523

commit: 9abe560

@wobsoriano

Copy link
Copy Markdown
Member

closing in favor of #10507

@wobsoriano wobsoriano closed this Oct 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants