-
Notifications
You must be signed in to change notification settings - Fork 934
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
1826 remove @mui/styles from project #1862
Conversation
@zxhmike @wdh2100 @ashfaqnisar @gregnb please take a look into this PR |
Hey @Fullbusters, Great Work Buddy. I have connected this PR to my project and everything worked great. Didn't find any visible issues with the component. Will push the fixes if i found any later on. Just had some lint issues, which I pushed a Fullbusters#1 to you're repo. |
I think we also have to update the readme, so the users should install tss-react along with the material ui packages |
@ashfaqnisar thanks, already update current pull request with your changes, thanks for fast review |
Hey @wdh2100, this looks great, can you go ahead merge this and we can push new changes if there are any issues in the future. |
Super excited to see this update, finally |
thanks 😄 It will have to be completely replaced later. |
I will release a new version soon |
@wdh2100 Hey was this released? still getting this when I migrated over to pnpm (from yarn) in v |
remove @mui/styles according to #1826