-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[DataGrid] Add use client
directive to the Grid packages
#11803
Conversation
Deploy preview: https://deploy-preview-11803--material-ui-x.netlify.app/ |
f553160
to
f1d5eee
Compare
282b1d8
to
93fe497
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
1 similar comment
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Do we need to add "use client" to the exported subcomponents, like |
Yes, we could, won't it apply to every other slot component too though? As a reference, I tried to use I'd propose adding it to the entry point Wdyt? |
Closes #11699
For the simple use cases when no function (or something non-serializable) is used in the App Router code using Data Grid, this PR will help to not force the users to use
use client
directive on their side.