Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Replace componentWillReceiveProps with componentDidUpdate #143

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jonthomp
Copy link

componentWillReceiveProps will be renamed UNSAFE_componentWillReceiveProps in react 17 and reults in console errors when run in React strict mode.

This PR replaces all use of componentWillReceiveProps with componentDidUpdate. While this should be sufficient, more optimizations might be found switching to some memoized functions.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant