React PropTypes warning with LayoutAnimation

[janicduplessis]

We are calling PropTypes manually in LayoutAnimation which now triggers a warning since React 16. https://facebook.github.io/react/warnings/dont-call-proptypes.html explains the warning but there doesn't seem to be an ideal solution in our case, including a new dependency to do these validations seems a bit overkill and writing these validations manually a bit too complex.

Opening this issue to discuss what would be the best situation here. Some ideas...

• Write the validation manually
• Use a validation library that is pretty much the react prop types checker without the warning like https://github.com/developit/proptypes
• Use console.ignoredYellowBox to silence the warning.
• Include the React secret in RN and use it to silence the warning.

cc @gaearon @sebmarkbage

[janicduplessis]

I'm personally leaning towards point 3 or 4, those are more hacky but let's say that "we know what we are doing" :)

[sebmarkbage]

We have an open issue of this internally. We're exposing an external API to do the type checking explicitly. facebook/react#9004

[MobiusHorizons]

is there a temporary work-arround? I had to update to 0.43-rc.2 to get FlatList because of unfixed android bugs in ListView but this bug makes development a pain. I guess it goes away in the production version of the app, but that's still a real pain in development.

Also if there is specific work that you would like a PR for, I'd be glad to do it, as I know you guys have a lot of work on your plate.

[janicduplessis]

You can use console.ignoredYellowBox (http://facebook.github.io/react-native/docs/debugging.html#warnings)

[janicduplessis]

Also I have a PR already to fix this, should get merged soon and then we can include the fix in the next rc patch.

[fungilation]

I just upgraded to yesterday-released version of RN 0.43.0 and I'm seeing these yellow box warnings.

So please tell me. Is this ok to ignore in production as the PropTypes wouldn't be checked in production? Or there's a more proper fix coming before I release my app to production on RN 0.43? (@janicduplessis your PR hasn't gone in?)

Otherwise, in dev I'd just add these to yellowbox ignore. Thanks for a heads up on what's going on with this as the last message here was 18 days ago.

[bvaughn]

Sorry for the temporary annoyance, folks! Andrew and I have been preparing the react 15.5.0 release for the last few days that will- among other things- provide a new "official" proptypes package to use for validation (as mentioned in facebook/react#9004).

Our plan is to publish an RC for all of the packages/add-ons (hopefully sometime this evening) so that people can do some test runs with react/react-dom. Presumably we could use this new standalone proptypes lib?

cc @acdlite in case I misspoke about timing WRT the proptypes lib

[gaearon]

The issue here is that 0.43 final was released yesterday but didn't include this fix: d9910a5. So looks like everyone in open source will get this stray warning now. 😞

cc @grabbou What can we do to get this fix out asap?

[bvaughn]

☹️ I didn't realize that 43 stable had gone out yet. I thought we were still on RC.

[gaearon]

For people encountering the warning: please feel free to ignore it for now. Sorry this slipped through.

[grabbou]

I will cherry pick this ASAP as soon as I get to work, approx 2 hours.

…

On Wed, Apr 5, 2017, 01:31 Dan Abramov ***@***.***> wrote: For people encountering the warning: please feel free to ignore it <http://facebook.github.io/react-native/docs/debugging.html#warnings> for now. Sorry this slipped through. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#12458 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACWcxnMAhGnSeedNXpCBD6HifP1irWORks5rstLkgaJpZM4MFZbR> .

[grabbou]

@bvaughn it goes stable at the end of the month, if there's anything that you are working that needs extra time, please let me know on Slack so we can better organise that 👍