Skip to content

Add support for MUI v7#10639

Merged
slax57 merged 11 commits intonextfrom
support-mui-v7
Apr 16, 2025
Merged

Add support for MUI v7#10639
slax57 merged 11 commits intonextfrom
support-mui-v7

Conversation

@djhi
Copy link
Contributor

@djhi djhi commented Apr 2, 2025

Problem

MUI v7 is out.

Solution

Follow #10655

  • Update peer dependencies to include MUI v7
  • Use MUI v7 in the ecommerce demo
  • Keep the tutorial in v6

How To Test

To make sure the demo builds despite the @ts-expect-error here:

  • copy the demo somewhere outside the monorepo
  • add resolutions just like we do with create-react-admin but targets MUIv7:
"resolutions": {
        "@emotion/react": "^11.14.0",
        "@emotion/styled": "^11.14.0",
        "@mui/icons-material": "^7.0.0",
        "@mui/material": "^7.0.0",
        "react": "^19.0.0",
        "react-dom": "^19.0.0",
        "react-router": "^7.1.3",
        "react-router-dom": "^7.1.3"
},
  • copy the monorepo packages in the demo node_modules
  • Build the demo

Additional Checks

  • The PR targets master for a bugfix or a documentation fix, or next for a feature

@djhi
Copy link
Contributor Author

djhi commented Apr 2, 2025

Question: should we update the create-react-admin template too? @fzaninotto @slax57 @erwanMarmelab?

Copy link
Contributor

@slax57 slax57 left a comment

Choose a reason for hiding this comment

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

Other than that I found no issues while testing, well done!

Regarding your question about create-react-admin, yes I believe we should update the version in the templates (but please in another PR, this one's already big enough 😅 )

djhi and others added 2 commits April 15, 2025 09:05
Co-authored-by: Jean-Baptiste Kaiser <jb@marmelab.com>
@slax57 slax57 added this to the 5.8.0 milestone Apr 16, 2025
@slax57 slax57 merged commit b2135e5 into next Apr 16, 2025
16 checks passed
@slax57 slax57 deleted the support-mui-v7 branch April 16, 2025 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RFR Ready For Review

Development

Successfully merging this pull request may close these issues.

2 participants