Remove use of "unsafe" component lifecycle methods

[schnogz]

Remove use of "unsafe" component lifecycle methods

[TmasLee]

Hi, I could look into this when I can!

[schnogz]

Awesome, thanks so much @sendmebiscuits!

Just comment on the issue with the areas that you're working on so we don't have overlapping work between people. Also, if you have any questions or need help testing, feel free to reach out to me or @plondon anytime!

[TmasLee]

Sure thing! I'll take care of those in a bit.

[TmasLee]

Quick question before I start touching anything, should code in the deprecated lifecycle methods be refactored based on how the React Docs suggest? Or just a complete deletion of the deprecated method?

[plondon]

Let's just start with removing the deprecated methods please

[TmasLee]

Hey guys, I finally had some time to go through the code. I found and deleted 1 componentWillMount and 1 componentWillReceiveProps in packages/blockchain-info-components/src/Carousels/Carousel/index.js and packages/blockchain-info-components/src/Form/SelectInput/index.js. I'll push the changes and make a pull request.

[prwelber]

Thanks @sendmebiscuits, we appreciate your help 👍

There's a bunch more in packages/blockchain-wallet-v4-frontend/src. Let me know if you want to tackle those as well.

[prwelber]

Work ongoing here: #876

[plondon]

Closing!