-
-
Notifications
You must be signed in to change notification settings - Fork 272
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
is there any plan to upgrade react dependency to 17 ? #118
Comments
Maybe some can help out here if they have ideas? |
Hello, Is this just a matter of adding
|
Is there any progress on updating to support react 17? |
Bump - we need this to correct an issue with legacy peer dependencies in Payload. Can we get the above PR merged in? Everything appears to work with React 17. |
it doesnt feel like the project is active, I am considering forking this... |
I would help test/contribute if you do @feddyups |
I would also help test/contribute. |
We are using it with react@18 and seems to work fine. Maybe just remove the peerDependencies restriction? |
there seem to be multiple forks already now with https://www.npmjs.com/package/react-diff-viewer-continued being the most popular so far. I think I'm considering switching to that fork for now, maybe the community should consider the same / agree on a fork |
Thanks @herndlm |
the library now depends on react 15 & 16. it doesn't work under react 17. I am wondering is there any plan to upgrade the dependency?
The text was updated successfully, but these errors were encountered: