Update deprecated lifecycle method

[DCKT]

Hello,

componentWillMount and componentWillReceiveProps will be removed in future version, it's required to use the new methods with UNSAFE_ prefix in order to use it.

[chetan-zalake]

@DCKT This didn't work when updated in my project. I still get those deprecated warnings. Grateful if you could help with this problem?
@ascoders Till when can we expect this to be merged?

[cristianoccazinsp]

Don't forget to update the .ts files

Can you do another find search for those two deprecated methods? I believe you've left a few out, reason the warnings are still there.

@ascoders can we get this merged asap once it's finished?

[Tamenze]

+1 @ascoders

[sachinshettigar]

@DCKT Instead of UNSAFE prefix we can replace deprecated lifecycle methods with componentDidMount and componentDidUpdate

[cristianoccazinsp]

@DCKT Instead of UNSAFE prefix we can replace deprecated lifecycle methods with componentDidMount and componentDidUpdate

This is not entirely true. Keep mind those methods are semantically different, so such change would need to be tested (e.g., didUpdate fires on every render regardless of props change)

[TPXP]

@DCKT Instead of UNSAFE prefix we can replace deprecated lifecycle methods with componentDidMount and componentDidUpdate

This is not entirely true. Keep mind those methods are semantically different, so such change would need to be tested (e.g., didUpdate fires on every render regardless of props change)

In this case, the componentDidUpdate compares previous props with the new ones and will not do anything if the centerOn prop did not change, so this logic can be safely moved to the componentDidUpdate method.

As for the componentWillMount method, the actions performed in this method can be moved in the constructor.

See my PR for this approach #127

[ArtemKolichenkov]

UNSAFE is not great, yeah.
PR #130 will handle that. (Also check out #129 if you're interested what's going on with this package)