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

Make necessary changes to update to material-ui v5 RC #42

Merged
merged 1 commit into from
Nov 2, 2021
Merged

Make necessary changes to update to material-ui v5 RC #42

merged 1 commit into from
Nov 2, 2021

Conversation

ShotSkydiver
Copy link
Contributor

The just-released material-ui RC-0 includes the new package names that material-ui is transitioning to -- material-ui to mui. This PR updates the packages to that new version and makes the necessary renames in the code.

@henry-young
Copy link

Shout out to @Mitch528 for publishing this branch to NPM until this PR is merged https://www.npmjs.com/package/@mitch528/mdi-material-ui

@stunaz
Copy link

stunaz commented Sep 19, 2021

@leMaik any change to release this for the compatibility of material-ui v5

@mkermani144
Copy link

This also fixes #43.

@olee
Copy link

olee commented Oct 7, 2021

What is preventing this PR from being merged? Would really like to properly update this dependency in our project

@olee
Copy link

olee commented Oct 15, 2021

Bump?

@Philipp91
Copy link

Here's how to use the fork without updating all the imports:

"mdi-material-ui": "npm:@mitch528/mdi-material-ui@^7.0.0",

@leMaik
Copy link
Member

leMaik commented Nov 2, 2021

@olee Sorry, this went below my radar. 😓
@Mitch528 Thank you for jumping in! 🎉 I'll release a new version shortly.

@leMaik leMaik merged commit 13e7e7e into TeamWertarbyte:mui-v5 Nov 2, 2021
@leMaik
Copy link
Member

leMaik commented Nov 2, 2021

@ShotSkydiver Thanks a lot for updating this 🙏

@JeremyGrieshop
Copy link

Awesome, thank you, my icon colors are back to normal now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants