-
Notifications
You must be signed in to change notification settings - Fork 423
fix(ui): Keep role select enabled with default org role #7567
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(ui): Keep role select enabled with default org role #7567
Conversation
🦋 Changeset detectedLatest commit: 101b80d The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 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 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
6b7589e to
20fa14b
Compare
📝 WalkthroughWalkthroughAdds a changeset entry for a patch release and fixes a bug where the role selector on the OrganizationProfile invite members page could be disabled when the default role was not present. The default-role initialization was moved from the top level of 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📜 Recent review detailsConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (2)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (28)
🔇 Additional comments (1)
Comment |
20fa14b to
871fea7
Compare
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
packages/ui/src/components/OrganizationProfile/InviteMembersForm.tsx
Outdated
Show resolved
Hide resolved
232dbb7 to
101b80d
Compare
Description
Fix
Now only sets the field value on mount, if the default role matches an item on
/rolesCleanShot.2026-01-08.at.18.15.30.mp4
Previously
When
organizationSettings.defaultRoledoes not exist on the options returned by/roles, the field gets initialized withroleField.setValue, but the option cannot be found, therefore it disables the select input:Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit
Bug Fixes
Tests
Chores
✏️ Tip: You can customize this high-level summary in your review settings.