add missing Dutch localization (nl-NL) for the profileSection.primaryButton#7711
add missing Dutch localization (nl-NL) for the profileSection.primaryButton#7711MaartenKoller wants to merge 7 commits intoclerk:mainfrom
Conversation
add missing primary button text
🦋 Changeset detectedLatest commit: c370ee5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
|
@MaartenKoller is attempting to deploy a commit to the Clerk Production Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughUpdated Dutch localizations and release metadata: 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.changeset/free-rivers-cheer.md:
- Line 2: The changeset currently marks "@clerk/localizations" as "major" but
this is a backward-compatible localization fix; update the changeset entry for
the package name "@clerk/localizations" to use "patch" instead of "major" so the
release tooling will perform a patch bump rather than a major version bump.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro
Run ID: f5708a85-0f7b-470e-8757-4b855a039686
📒 Files selected for processing (1)
.changeset/free-rivers-cheer.md
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 bijwerken' in Dutch.
https://clerk.com/docs/nextjs/reference/components/organization/organization-profile
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 testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit