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

Warning about componentWillReceiveProps #62

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

Conversation

NouraldinS
Copy link

References #61

As mentioned in React docs here componentWillReceiveProps is being deprecated, and should be either renamed to UNSAFE_componentWillReceiveProps or be replaced with componentDidUpdate.

References jsdir#61 

As mentioned in React docs [here](https://reactjs.org/docs/react-component.html#unsafe_componentwillreceiveprops) `componentWillReceiveProps` is being deprecated, and should be either renamed to `UNSAFE_componentWillReceiveProps` or be replaced with `componentDidUpdate`.
cube-dan added a commit to perfectcube/react-ladda that referenced this pull request Nov 14, 2019
…/react-ladda.git#master“ will work out of the box. this is a stopgap until pr jsdir#62 is finalized and released on npm by jsdir/react-ladda
@guilhermealles
Copy link

Is anyone still maintaining this project? I see no reason not to merge this PR.

@edraghiciu-vks
Copy link

^ Any updates on this?

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