-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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: Prevent the Playground from crashing on a form error. #3164
Conversation
@nickgros Have you verified that this will handle all the uses cases you've indicated will be fixed? |
Co-authored-by: Heath C <51679588+heath-freenome@users.noreply.github.com>
Yep, and just added a few more. The Form component is still going to throw an error in user code, but in those cases, users should use an error boundary themselves, or open a new issue/PR and clearly define what the behavior should be. |
Fixes #3132, #3095, #2787, #2277, #2179, #2170, #2163, #1762, and #1002;
Potentially fixes other issues that I couldn't find.