-
Notifications
You must be signed in to change notification settings - Fork 192
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
Dependency resolution problems #187
Comments
Hey acrois, |
@ErikBaer I wasn't able to resolve it cleanly by doing it. Maybe I am doing it differently. Specifically what did you change in package.json in your project in order to fix the dependencies? Can you show all the material-ui related entries in your package.json for comparison? Thanks! |
@acrois I am not 100% confident in this update, but this got my initial install working: I had to use --force to get it to take. |
Funny, cloning & running this starter-app repo works: https://github.com/storefront-foundation/react-storefront-starter-app Something about that one's |
@TonyMakis Thank you, worked for me as well. |
Thanks @TonyMakis it did work. I think that means the issue exists somewhere between 8.17.4 and current version 8.18.3. When I cloned it, the issue was non-existent. However, you can not delete both the package-lock.json and node_modules and having a fresh dependency resolution. You can trick it to work by using "--legacy-peer-deps" like "create-react-storefront" does but then the issue with material-ui pops up when you visit a category page which is kinda the real issue underlying it all. All in all, seems to appear that in order to install or update those dependencies you must always force them to install, but this is unfortunately not documented anywhere. The material UI components have seemingly really complex dependencies, we may be using "lab" features, etc. so someone who has experience with updating the dependencies should chime in on this. I don't really see an issue using features from libraries like that but I do see a need in sharing the knowledge of how the dependencies can be updated (in the case like this). @randy-parcel-ag I was able to apply this after creating an app using: The dependencies just seem to be the exact same but produce different results. https://github.com/storefront-foundation/create-react-storefront/blob/master/lib/create-react-storefront-internal.js#L69 uses --legacy-peer-deps and this is also a relatively new change as well. Notice: someone decided they are forcing all users to use --registry=https://npm-proxy.fury.io/moovweb/ argument during create-react-storefront. I don't know if this is interfering with the availability of certain packages or indexes but how do you audit this?? For now we're at the mercy of the third-party deps. Let's --force our way through and hope it doesn't break something else. We'll have to wait until someone more knowledgeable about the dependencies comes along to clarify what's going wrong and resolve the issues. |
I found here that this issue comes from @material-ui/lab's version. So I my package.json file, I changed
By simply removing the ^ and running |
Bug report
Describe the bug
A fresh install of the master branch will yield a broken starter app. There are dependency resolution errors due to the recent update to material-ui
I am testing a fix, may submit a PR if successful.
To Reproduce
Follow README instructions to create a new app. Clear package-lock.json and npm install to attempt to resolve the dependencies.
Simply:
npm start
Attempt to load a subdirectory using mock data. You will be met with an error.
You can also attempt:
npm update
on a fresh clone of this repo @ masterExpected behavior
Dependencies install correctly.
Screenshots
System information
Additional context
I found these two releases interesting:
https://github.com/mui-org/material-ui/releases/tag/v4.11.2
https://github.com/mui-org/material-ui/releases/tag/v4.11.4
The text was updated successfully, but these errors were encountered: