-
Notifications
You must be signed in to change notification settings - Fork 193
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 unsafe lifecycles #134
Comments
Awesome. Thank you for maintaining this lib. 🙌 |
Hmm. I'm still getting the warning, but only one this time:
|
Interesting, surprised they are doing away with those ones too. Okay, will reopen and fix this up soon thanks for the detail. |
Is it possible to revisit this issue soon? I would like to use this package but I am not sure it is being actively maintained. Thanks! |
Agreed, I see #151 for |
We are seeing this a lot and are wondering when a fix will be available? |
also update codemirror manually still getting a `componentWillReceiveProps has been renamed, and is not recommended for use` warning; being tracked at scniro/react-codemirror2#134
also update codemirror manually still getting a `componentWillReceiveProps has been renamed, and is not recommended for use` warning; being tracked at scniro/react-codemirror2#134
Is it possible to merge #151? Or get a fix for this? Would really like to use this library but reluctant since componentWillReceiveProps will not work at all after React 17 release |
fyi #151 will not work in react 17 either. |
I don't know if they said something else somewhere but their blog says
|
@diegohaz @nickbeaulieu @ldeveber @kaniseo @Janpot I am a lot shorter on time these days as when I started this project. Codemirror & React APIs are moving to quickly for me to keep atop of for the day-to-day. I am looking for a co-maintainer of this project. Please contact me directly if you are interested. Thank you for understanding. |
This should be resolved with #176 and v6.0.1 / v7. |
When using
react-codemirror2
within<StrictMode />
or with Reactv16.9.0
, it shows a few warnings on the console:The text was updated successfully, but these errors were encountered: