Skip to content

Update nl-NL.ts#7711

Open
MaartenKoller wants to merge 2 commits intoclerk:mainfrom
MaartenKoller:patch-2
Open

Update nl-NL.ts#7711
MaartenKoller wants to merge 2 commits intoclerk:mainfrom
MaartenKoller:patch-2

Conversation

@MaartenKoller
Copy link
Contributor

@MaartenKoller MaartenKoller commented Jan 30, 2026 •
edited by coderabbitai bot
Loading

add missing primary button text

Description

Currently a button has no text in my localization. Adding this should fix that.

The original component says 'Update profile' which translates to 'Profiel aanpassen' in Dutch.
https://clerk.com/docs/nextjs/reference/components/organization/organization-profile

image

You can test it using the nl-NL localization and use the component. In the current version the button should then look like it does above. After this pull request merges it should populate with the correct text.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other: missing string

Summary by CodeRabbit

  • Bug Fixes
    • Restored missing Dutch label for the profile section button — it now shows "Profiel aanpassen" so users see the proper action text.

✏️ Tip: You can customize this high-level summary in your review settings.

@MaartenKoller
Update nl-NL.ts
ff529cf 
add missing primary button text
@changeset-bot
Copy link

changeset-bot bot commented Jan 30, 2026 •
edited
Loading

⚠️ No Changeset found

Latest commit: 36808b6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@vercel
Copy link

vercel bot commented Jan 30, 2026

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

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

The change updates a localization file for the Dutch (Netherlands) language. Specifically, the primaryButton property within the profileSection object in packages/localizations/src/nl-NL.ts is modified from an empty string to the Dutch text "Profiel aanpassen". This is a single-line modification affecting the localized user interface text for the Dutch language pack.

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Update nl-NL.ts' is vague and generic—it describes the file being modified but not the actual meaningful change (adding missing Dutch localization for a profile button). Consider revising the title to be more descriptive, such as 'Add missing Dutch translation for profile update button' or 'Update nl-NL localization for OrganizationProfile component'.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

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

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

core-3 localizations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MaartenKoller