Migrate away from componentWillReceiveProps

[havardh]

Hi. 👋

By running in react strict-mode, I noticed that that react-vis relies on the now deprecated componentWillReciveProps.

With a quick search in the repo I found the following usages (examples are omitted):

• ArcSeries
• ThreeMap
• makeFlexible
• XyPlot

These should be easily resolvable by moving to either getDerivedStateFromProps or componentDidUpdate or memoization as detailed here and here.

Are there any plans for doing or concerns about doing these changes?

If these changes makes sense for the project, I could make a PR for them 👍

[mcnuttandrew]

Hey that sounds like great work to be doing! I would love to see a PR to that effect

[taltultc]

+1

[fedorov-xyz]

+1

[TimoPurzner]

Please fix this or the Project will soon stop working :)
Im getting the warning:

Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run `npx react-codemod rename-unsafe-lifecycles` in your project source folder.

Please update the following components: FlexibleXYPlot

[testacode]

Same here please!

[yukoliesh]

Is there any update on this?

[tp00012x]

Just close this library if you guys are not going to maintain it well

[av-k]

The project exists 8 years, the issue exists 5+ years, is something happening here?