novalabio/react-native-maps-super-cluster

Warnings in library

Opened this issue · 4 comments

plungarini commented

Hello guys I'm receiving those two warnings. Can you update the library and let me know please?

Warning: componentWillUpdate has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.

* Move data fetching code or side effects to componentDidUpdate.
* Rename componentWillUpdate to UNSAFE_componentWillUpdate 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: %s, ClusteredMapView

Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.

* Move data fetching code or side effects to componentDidUpdate.
* If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state        
* 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: %s, ClusteredMapView
39otrebla commented

@plungarini we have a major refactoring in progress which does include switching to Functional Components. That should fix all those warnings. I'll keep you posted.

  • 🎉5
Maqsim commented

Hey @39otrebla Thanks for you work. I am also waiting for new refactoring

Badra00 commented

Hey @39otrebla any news on this?

  • 👍3
dmackinn commented

This package hasn't been updated in 5+ years, is this still planned on being refactored?

  • 😕1