react-native-autocomplete-input icon indicating copy to clipboard operation
react-native-autocomplete-input copied to clipboard

Fix ViewPropTypes warning

Open ajenkins opened this issue 3 years ago • 9 comments
trafficstars

After upgrading to React Native 0.68, I started seeing this warning: ViewPropTypes will be removed from React Native. Migrate to ViewPropTypes exported from 'deprecated-react-native-prop-types'.

Personally, I don't love the suggested solution, but I don't know what the alternative is (I asked about better alternatives here).

I based this PR off of a similar PR for react-native-maps.

It's unfortunate that this fix requires adding the first and only dependency to this package, but without adding deprecated-react-native-prop-types to dependencies, anyone who wants to use this package would need to have their own copy of deprecated-react-native-prop-types installed, which would be annoying.

ajenkins avatar May 15 '22 14:05 ajenkins

I believe this fixes #222

ajenkins avatar May 15 '22 15:05 ajenkins

Update: Got a reply from someone on the React team about the "right" way to fix this. Apparently they want everyone to start using Flow or Typescript instead of relying on prop types.

Migrating to Flow or Typescript would be a much bigger change though, and probably isn't worth doing just to fix this one warning.

ajenkins avatar May 15 '22 18:05 ajenkins

Personally, I would greatly appreciate for this pr to be merged since this is the only package I have left which creates this warning :)

Thanks for the great work!

DavidAmyot avatar May 19 '22 17:05 DavidAmyot

Hi, Any update on this?

mak12 avatar Jun 08 '22 17:06 mak12

Hi, Any update on this?

I wish! @mrlaessig Any chances of this getting merged?

Thanks :)

DavidAmyot avatar Jun 08 '22 17:06 DavidAmyot

This library is now raising an error (was a warning), when opgrading to react native 0.69. Merging this PR now it's pretty important.

anija avatar Jul 05 '22 10:07 anija

Any news on this PR ? It is breaking RN 0.69.

thomasviaud avatar Jul 21 '22 13:07 thomasviaud

If some people need that urgently, you can still use https://www.npmjs.com/package/patch-package. Working well applying changes in this PR. Not ideal but can save a build.

thomasviaud avatar Jul 22 '22 08:07 thomasviaud

having the same issue. would be great to merge this!

ericmager avatar Aug 09 '22 18:08 ericmager