Skip to content

Conversation

@MaartenKoller
Copy link
Contributor

@MaartenKoller MaartenKoller commented Jan 30, 2026

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
    • Fixed missing text label for the profile section button in Dutch language, ensuring it now displays correctly to users.

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

add missing primary button text
@changeset-bot
Copy link

changeset-bot bot commented Jan 30, 2026

⚠️ No Changeset found

Latest commit: ff529cf

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

📝 Walkthrough

Walkthrough

The change updates a localization file for the Dutch (Netherlands) language. Specifically, the primaryButton property within the profileSection object in the nl-NL.ts file 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 | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Update nl-NL.ts' is vague and generic, lacking specificity about what was actually changed in the file. Use a more descriptive title like 'Add missing Dutch localization for profile update button' to clearly convey the change purpose.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant