Skip to content

[charts] Add "use client" directive to every public component and hook#14578

Merged
flaviendelangle merged 1 commit intomui:masterfrom
flaviendelangle:use-client-charts
Sep 11, 2024
Merged

[charts] Add "use client" directive to every public component and hook#14578
flaviendelangle merged 1 commit intomui:masterfrom
flaviendelangle:use-client-charts

Conversation

@flaviendelangle
Copy link
Member

@flaviendelangle flaviendelangle commented Sep 11, 2024

Follow up on #14562. Part of #9833.

@alexfauquette @JCQuintas I let you check if some components or hooks can work and be valuable in RSC 👍

@flaviendelangle flaviendelangle added the scope: charts Changes related to the charts. label Sep 11, 2024
@flaviendelangle flaviendelangle self-assigned this Sep 11, 2024
@flaviendelangle flaviendelangle changed the title [charts] Add use client directive to every public component and hook [charts] Add "use client" directive to every public component and hook Sep 11, 2024
@mui-bot
Copy link

mui-bot commented Sep 11, 2024

Deploy preview: https://deploy-preview-14578--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 0f51bdc

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 11, 2024

CodSpeed Performance Report

Merging #14578 will not alter performance

Comparing flaviendelangle:use-client-charts (0f51bdc) with master (1939c1c)

Summary

✅ 3 untouched benchmarks

@flaviendelangle flaviendelangle merged commit 4f2a244 into mui:master Sep 11, 2024
@flaviendelangle flaviendelangle deleted the use-client-charts branch September 11, 2024 12:40
@@ -1,3 +1,4 @@
'use client';
Copy link
Member

@oliviertassinari oliviertassinari Sep 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure here: mui/material-ui#41956 (comment).

Edit: no, we are good mui/material-ui#43778.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: charts Changes related to the charts.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants