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

componentWillReceiveProps warning with React 16.9 #569

Open
alexbensimon opened this issue Aug 12, 2019 · 18 comments
Open

componentWillReceiveProps warning with React 16.9 #569

alexbensimon opened this issue Aug 12, 2019 · 18 comments

Comments

@alexbensimon
Copy link

Hello,

After updating react and react-dom to v16.9.0, this warning appears:

react-dom.development.js:11494 Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

* Move data fetching code or side effects to componentDidUpdate.
* If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state
* Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run `npx react-codemod rename-unsafe-lifecycles` in your project source folder.

Please update the following components: Motion

According to https://reactjs.org/blog/2019/08/08/react-v16.9.0.html#renaming-unsafe-lifecycle-methods, the componentWillReceiveProps is now deprecated and will be removed from the API on the next major version.

@klimashkin
Copy link

klimashkin commented Aug 28, 2019

It feels like the project has been abandoned by the owners.
There was a PR #534, but didn't get through

@juicylevel
Copy link

2020 year..

@nkbt
Copy link
Collaborator

nkbt commented Jan 21, 2020

Yeah @juicylevel, I wish I could drop all the things of my life now (as in work, surfing, divorce, girlfriend) and do all the opensource...

On a side note: I've added a reminder to publish new version of react-motion which should have this fixed (at least temporarily). I'll let everyone know when it is published. You can help by teting current master branch in your real-world project and see if anything breaks.

@jfroffice
Copy link

Any update on this ?

@comp615
Copy link

comp615 commented Apr 17, 2020

When is the reminder to publish set for? Are there any other maintainers who might be able to help you out and publish this in the meantime? I know it's annoying to have people nagging, but getting the newer React support updated would be super helpful! Thank you!

@fabiomcosta
Copy link

Ah interesting... so sounds like the change is already there and we just need a new version published to npm?
If the maintainers give me the permissions I'll happily do that.

@andresmatasuarez
Copy link

yes, please!

@LiteSoul
Copy link

LiteSoul commented Feb 2, 2021

...and now it's 2021 🥺

@RaddishIoW
Copy link

Someone has forked react-motion to fix this issue: see this thread

@onionhammer
Copy link

Perhaps other projects should reference the fork

@RaddishIoW
Copy link

@nkbt Any update on this? If there's a need for help with maintaining this, I'm sure there's competent people here willing to help...

@nkbt
Copy link
Collaborator

nkbt commented Feb 16, 2021

I am still getting notifications and emails, btw =)
It is not just the matter of approving PR and pushing new release unfortunately (oh I wish it would be)

Here is some context. It is hard for me to test react-motion in real-world product as I no longer work with any projects using react-motion. So the only way for me now would be to use demo examples only (I remember having a few edge cases beyond those). Also I never really used or worked with anything except main Motion component (plus core), so have basically no idea about things like staggered motion, etc. There was partially completed refactoring which we did not get finished as contributor vanished and I did not have any spare time to finalise all the work myself, thus releasing new version to the public was blocked (still is, as it turns out). Should have reverted all back in hindsight...
@RaddishIoW There were few cases people jumping on and helping out with the library one or a few times and then leaving, so not like there was anyone specific committed to do full on support work on it.
Generally it's quite hard to squeeze opensource in between all the life things like family, baby, travelling, work, personal projects, unless it is used in your work projects and you have ability to contribute as part of your day job (which is not the case for me anymore for quite a while now).
I'll see if I can do anything, but cannot promise much at this point. I want to make sure it is in shape to safely do patch releases without breaking changes (check past PRs and revert unfinished work from master). Then it would be in a maintenance mode (just keep updating critical deps, etc).

@atom258
Copy link

atom258 commented Sep 23, 2021

2021

@JohnRoseDev
Copy link

JohnRoseDev commented Nov 22, 2021

How can an amazing project with 20k stars, conf presence (react europe talk), and a fantastic api be abandoned like this by the owner(s)? Completely understand if they don't have time anymore for the project, no blame in that, but couldn't one try in that case to delegate some ownership so someone can simply publish a new version of this package since the necessary code change is already there?

@catamphetamine
Copy link

2022

@GPrimola
Copy link

2023

Repository owner deleted a comment from theformer Jan 18, 2023
Repository owner deleted a comment from theformer Jan 18, 2023
@datplt
Copy link

datplt commented Jan 29, 2024

2024

@theformer
Copy link

theformer commented Jan 29, 2024 via email

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

No branches or pull requests