Conversation
add missing primary button text
|
|
@MaartenKoller is attempting to deploy a commit to the Clerk Production Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThe change updates a localization file for the Dutch (Netherlands) language. Specifically, the 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ 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 reviews:
finishing_touches:
unit_tests:
enabled: trueTry it out by using the 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. Comment |
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
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
✏️ Tip: You can customize this high-level summary in your review settings.