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

Update @emotion/* dependencies #151

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

MatiasCiccone
Copy link

@MatiasCiccone MatiasCiccone commented Feb 14, 2022

In our project, we had a compatibility issue with the current emotion versions installed in the project and other dependencies after upgrading to MUI 5 and React 17.
I would like to keep emotion updated for this library, do you agree?
Meanwhile, we are working with a fork published on NPM with the new dependencies

PD. I also noticed that classnames could be removed from the project and use the emotion built-in utility for classes

src/styles.ts Outdated Show resolved Hide resolved
@HaleyWardo
Copy link

@praneshr is there anyway we can get this change in? I am also running into this issue with the new MUI V5 update and this is blocking me from moving forward with releasing. Unfortunately, A simple resolution won't work because of the package rename. 🙏

Co-authored-by: Haley <haley.ward@lifeomic.com>
Copy link

@HaleyWardo HaleyWardo left a comment

Choose a reason for hiding this comment

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

Thank you for making that update! Also, It looks like there are merge conflicts.

package.json Outdated Show resolved Hide resolved
@dargmuesli
Copy link

You might wanna create that PR against https://github.com/Aeolun/react-diff-viewer-continued

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.

3 participants