-
Notifications
You must be signed in to change notification settings - Fork 404
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
React 16.3+ compatibility: replace componentWillReceiveProps with componentDidUpdate #141
base: master
Are you sure you want to change the base?
Conversation
@@ -73,7 +73,7 @@ module.exports = class Reader extends Component { | |||
this.initiateLegacyMode() | |||
} | |||
} | |||
componentWillReceiveProps(nextProps) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's not just about renaming the lifecycles method, and also, componentWillReceivePros is replaced by getDerivedStateFromProps, not componentDidUpdate
Why was this closed? Even if not the best approach, the fix would work and remove any warning message. Please consider reopening and merging. |
Btw. https://github.com/kybarg/react-qr-scanner works with react 16.3+ |
But it doesn't seem to work fine on MacOs / iOS. |
Hey guys any plans on merging this or fixing the |
Hello, |
same here |
Replacement looks to be straightforward.