redirect.js replace UNSAFE_componentWillMount with componentDidMount

[orenklein]

Any reason not to replace UNSAFE_componentWillMount with componentDidMount?

[ashishnumino]

Even I am facing same issue? Any timelines to fix this issue?

[zakriabilal]

Getting the same issue over here.

Any possible quick fixes that can be implemented?

[janoist1]

+1

[mjrussell]

Oops misread at first.

Why does this need a fix? The behavior is different between the two. Feel free to try a PR but I think you'll find out why its important to use WillMount instead of DidMount

[jlimas]

It's because of this warning when using this library @mjrussell I have a fresh React installation only with this library and started getting this warnings on dev tools.

Warning: Using UNSAFE_componentWillMount in strict mode is not recommended and may indicate bugs in your code. See https://reactjs.org/link/unsafe-component-lifecycles for details.

* Move code with side effects to componentDidMount, and set initial state in the constructor.

Please update the following components: Redirect

[mjrussell]

As I said earlier. I'm not sure I can maintain the same behavior/invariants but I welcome a PR that passes the test suite.