-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Migrate Checkbox to Chakra Native Component #8204
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
Conversation
✅ ethereum-org-website-dev deploy preview ready
|
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.
Solid work on this @marcellamalune. Thanks so much for this PR. I left a few comments to cleanup the old props and logic we had before. The rest is great.
Co-authored-by: Pablo Pettinari <[email protected]>
Co-authored-by: Pablo Pettinari <[email protected]>
|
@pettinarip thank you for your feedbacks! I applied all your suggestions. |
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.
Awesome @marcellamalune! 🎉 I've just added some cleanup stuff. Will merge it soon.
Migrate Checkbox to Chakra Native Component
Description
Related Issue
#6374