Skip to content
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

React 19 support #15342

Merged
merged 32 commits into from
Nov 22, 2024
Merged

React 19 support #15342

merged 32 commits into from
Nov 22, 2024

Conversation

arminmeh
Copy link
Contributor

@arminmeh arminmeh commented Nov 8, 2024

Closes #13356

To run locally, after checkout run
pnpm use-react-version rc && pnpm use-material-ui-v6 (since v5 does not support r19)

All checks are ✅
building the packages works fine
some tests are still failing (would appreciate if someone from @mui/explore would look at the one with datepicker)

Test updates done in a same way and for the same reasons as in
mui/material-ui#43155

@arminmeh arminmeh added component: data grid This is the name of the generic UI component, not the React module! component: pickers This is the name of the generic UI component, not the React module! component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! component: charts This is the name of the generic UI component, not the React module! React 19 support PRs required to support React 19 labels Nov 8, 2024
Copy link

codspeed-hq bot commented Nov 8, 2024

CodSpeed Performance Report

Merging #15342 will improve performances by 12.5%

Comparing arminmeh:react-19 (2e5e2f5) with master (6fc6978)

Summary

⚡ 1 improvements
✅ 5 untouched benchmarks

Benchmarks breakdown

Benchmark master arminmeh:react-19 Change
ScatterChartPro with big data amount 200.4 ms 178.1 ms +12.5%

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Nov 15, 2024
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Nov 18, 2024
@arminmeh arminmeh changed the title [WIP] React 19 support React 19 support Nov 19, 2024
@arminmeh arminmeh marked this pull request as ready for review November 19, 2024 09:20
@arminmeh arminmeh requested a review from a team November 19, 2024 10:28
@arminmeh arminmeh force-pushed the react-19 branch 2 times, most recently from e5156af to 3403086 Compare November 20, 2024 09:35
scripts/useMaterialUIv6.mjs Outdated Show resolved Hide resolved
packages/x-tree-view/package.json Outdated Show resolved Hide resolved
@arminmeh arminmeh added needs cherry-pick The PR should be cherry-picked to master after merge v7.x labels Nov 21, 2024
@arminmeh
Copy link
Contributor Author

I also did a manual test
Installed next 15 and react 19 rc1 with our packages
Used examples from "Getting started" pages for all component and all their versions (community, pro, premium)
All of them rendered without any warnings/errors

Copy link
Member

@LukasTy LukasTy left a comment

Choose a reason for hiding this comment

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

Thank you for your and everyone involved effort on this one. 👍
LGTM, but I'm confirming without manual testing, given the existing testing done. 🙈

steps:
- run:
name: Install @mui/material@next
name: Install @mui/material v6
Copy link
Member

Choose a reason for hiding this comment

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

Nice correction. 👍 🙈

test/README.md Outdated Show resolved Hide resolved
@arminmeh arminmeh merged commit cac5df0 into mui:master Nov 22, 2024
24 checks passed
Copy link

Cherry-pick PRs will be created targeting branches: v7.x

github-actions bot pushed a commit that referenced this pull request Nov 22, 2024
Signed-off-by: Armin Mehinovic <4390250+arminmeh@users.noreply.github.com>
Co-authored-by: flavien <flaviendelangle@gmail.com>
Co-authored-by: Andrew Cherniavskii <andrew.cherniavskii@gmail.com>
Co-authored-by: Lukas Tyla <llukas.tyla@gmail.com>
@arminmeh arminmeh mentioned this pull request Nov 22, 2024
@arminmeh arminmeh deleted the react-19 branch December 2, 2024 18:44
LukasTy added a commit to LukasTy/mui-x that referenced this pull request Dec 19, 2024
Signed-off-by: Armin Mehinovic <4390250+arminmeh@users.noreply.github.com>
Co-authored-by: flavien <flaviendelangle@gmail.com>
Co-authored-by: Andrew Cherniavskii <andrew.cherniavskii@gmail.com>
Co-authored-by: Lukas Tyla <llukas.tyla@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: charts This is the name of the generic UI component, not the React module! component: data grid This is the name of the generic UI component, not the React module! component: pickers This is the name of the generic UI component, not the React module! component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! needs cherry-pick The PR should be cherry-picked to master after merge React 19 support PRs required to support React 19 v7.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[data grid] React 19 support
5 participants