-
Notifications
You must be signed in to change notification settings - Fork 46
React 16.9 warning - componentWillReceiveProps #42
Comments
Appreciate it, we are moving this project to a package under another project. With that its being rewritten with hooks and this warning will be removed. |
@GoPro16 what project will contain this? I'm leery of taking on this as a dependency if it's likely to move to a project that contains other pieces I don't need. Thanks! |
@wrogersTBS we would just be moving this to a mono repo https://github.com/Availity/availity-react that helps us manage a bunch of independent libraries. This move simply reduces are support cost and should not impact the usage of this library. |
@robmcguinness when would you plan to move to another repo and rewrite into use effect? |
@epotockiy not sure but I will make inquiry. |
@epotockiy you can watch #33 for any progress made to that. We are working on getting our components to compile and then will most likely revisit this issue when we make the move to the other react mono repo. Exact timeline is not known but feel free to submit a PR if you want to give it a whirl |
Is there progress on this warning? |
2 similar comments
Is there progress on this warning? |
Is there progress on this warning? |
Still hitting this warning. Any status updates? |
Is there progress on this warning? |
It seems the owner is no more or has left the repository to perish. |
any update on this warning? |
i am working on this. |
Any update on progress? |
@dvparth see Availity/availity-react#952 for some more details. @jordan-a-young has worked on converting this package to use hooks and TS and has proposed moving it into our |
Just an FYI... with react 16.9 I see the warning below. Seems componentWillReceiveProps is used in one place in the src (and a few places in tests).
The text was updated successfully, but these errors were encountered: