-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add null type to Radiogroup value
prop.
#5896
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
@LFDanLu Hi, I have a question that's unrelated to this PR. does every PR deploy the associated storybook somewhere? I think I saw some long domain names that include The reason why I ask this is that I thought it would be nicer to share the deployed storybook for a PR (if exists) than using |
@sookmax For contributor PRs we don't automatically run the storybook build for security reasons hence why you might notice the occasional comment from a team member of "GET_BUILD" that triggers it manually. |
Build successful! 🎉 |
## API Changes
unknown top level export { type: 'any' } |
Closes #5871
✅ Pull Request Checklist:
📝 Test Instructions:
Story
Controlled RadioGroup
http://localhost:9003/?path=/story/react-aria-components--radio-group-controlled-example&providerSwitcher-express=false&strict=true
🧢 Your Project: