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
[system] Fix handling of null-ish values #24527
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.
👍
Details of bundle changes.Comparing: 4295444...52f6592 Details of page changes
|
I'm waiting for codesandbox ci build to complete to test the change, we need to make sure that it works end-to-end. For some reasons, I couldn't reproduce the error in Next.js's docs. |
Ok, it's fixed. before: https://codesandbox.io/s/box-issue-with-undefined-forked-hxp13 |
For developers that want to use this patch until we release v4.11.3, follow https://github.com/mui-org/material-ui/blob/next/CONTRIBUTING.md#how-can-i-use-a-change-that-wasnt-released-yet |
To use commit as a dependency follow instructions here https://ci.codesandbox.io/status/mui-org/material-ui/pr/24527, section E.g.
|
Closes #24519. It's targetting
master
because it fixes an issue with a third-party dependency we have no control over: Chrome and it throws an exception. I will apply it tonext
too afterward.