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

Update RevealBase.js #70

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

Conversation

andrewlsimplisafe
Copy link

Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps for compatibility with React 17

Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps for compatibility with React 17
@apepper
Copy link

apepper commented Oct 24, 2019

Fixes #66.

@acato-dev
Copy link

@rnosov Is it possible for you to find some time to approve and merge this update? Fixes numerous warnings due to spec changes in the react base.

@andrewlsimplisafe
Copy link
Author

The maintainer of this repo is not actively maintaining this repository. I would suggest forking this repo, fixing and then uploading to a private package repo until the maintainer is able to update/rewrite. See this issue #69

@apepper
Copy link

apepper commented Jan 21, 2020

@andrewlsimplisafe : There is a second componentWillReceiveProps in the code base. See

componentWillReceiveProps({ query }) {
.

@sauravhiremath
Copy link

sauravhiremath commented Jul 25, 2020

Hey, @andrewlsimplisafe are you still working on this?

@MichaelDeBoey
Copy link

Ping @rnosov

@ravichopra0107
Copy link

ping @noelzubin

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.

None yet

7 participants