react-tiny-virtual-list icon indicating copy to clipboard operation
react-tiny-virtual-list copied to clipboard

Published 20 hours ago • verified publisher icon clauderic

Mark componentWillReceiveProps UNSAFE

Open y6nH opened this issue 6 years ago • 2 comments

See #66. This is the minimum change required for React 17 compatibility. The warning thrown by React 16, for reference:

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.

y6nH avatar Nov 08 '19 13:11 y6nH

This has to be fixed error: Identifier 'UNSAFE_componentWillReceiveProps' is not in camel case (camelcase) at src/index.tsx:173:3:; guess that adding a eslint disable next line?

marsonmao avatar Dec 04 '19 09:12 marsonmao

Now it's really broken. I suspect we have to update dependencies.

A one-liner is never a one-liner.

y6nH avatar Dec 05 '19 12:12 y6nH