-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
I'm using mui v5.11.0, After install that some errors are occurred in my project #35480
Comments
@FjVillar i'm using mui v5.11.0, but errors are occurred |
Please share a repository that we can reproduce. |
In my library I'm getting reports that my Apparently, this no longer works import debounce from '@mui/material/utils/debounce'; I'm pushing an emergency fix with this import { debounce } from '@mui/material/utils'; But I'm going to get a log of complaints while devs install incompatible versions |
Hi @siriwatknp, Reproduction: git clone https://github.com/garronej/react-dsfr-next-demo
cd react-dsfr-next-demo
yarn
yarn dev The repo is pinned on Reason of the bug: Since |
@garronej Thanks! We are figuring out what happened. |
See #35489 for an explanation of what happened. Long story short - the current format is the one we had before 5.10.3. So it should work the same way as the older versions. @garronej A brand new Next.js project with Material UI 5.11.0 works fine. Some custom config value likely causes problems after upgrading. I'm investigating it. https://mui.com/material-ui/guides/minimizing-bundle-size/#option-one-use-path-imports
@KevinVandy You're not supposed to use such deep imports. As noted in https://mui.com/material-ui/guides/minimizing-bundle-size/#option-one-use-path-imports, we only support up to 2nd level of subpaths. @garronej It seems that react-dsfr suffers from the same issue: https://github.com/codegouvfr/react-dsfr/blob/main/src/mui.tsx#L14 |
With react DSFR this is by design, the lib does not provide ESM distribution (to be able to import assets internally) and explicitely provides setup instructions to make it work. |
I meant that it has the same issue as @KevinVandy's code - it imports from a path three levels deep. |
Ah ok! Thank you very much. Sorry for the falsly alarmin report and thank you for the quick explaination. |
It sounds like @KevinVandy and @garronej issues are answered but might not be related to the main issue. Waiting for @rajagopal1997 for more details. |
yeah @siriwatknp it's solved my problem, im installing npm i @mui/material --force after installing forcely errors are resolved... |
Thanks for the info. I'm closing this issue then. |
Duplicates
Latest version
Steps to reproduce 🕹
Link to live example:
Steps:
1.Check my Images
2.how to solve that issue
Current behavior 😯
No response
Expected behavior 🤔
No response
Context 🔦
No response
Your environment 🌎
npx @mui/envinfo
The text was updated successfully, but these errors were encountered: