inProgress-team / react-native-meteor

Meteor Reactivity for your React Native application :)
MIT License
693 stars 210 forks source link

[Deprecation] NetInfo Dependency Moved #342

Open pkcwong opened 5 years ago

pkcwong commented 5 years ago

NetInfo has been moved from react-native to react-native-community. The original import is deprecated and will soon be removed.

https://github.com/inProgress-team/react-native-meteor/blob/7d00d1d0d499c5dddb30c4573dbe812027c29128/src/Meteor.js#L1

The current library throws a warning:

Warning: NetInfo has been extracted from react-native core and will be removed in a future release. It can now be installed and imported from '@react-native-community/netinfo' instead of 'react-native'. See https://github.com/react-native-community/react-native-netinfo

Propose to update the dependency to react-native-community in order to suppress the warning.

React Native NetInfo Website: https://facebook.github.io/react-native/docs/netinfo.html

musemind commented 4 years ago

With React Native 0.61 (used in Expo SDK 36) this causes an error.

nikhilgoswami commented 4 years ago

https://github.com/inProgress-team/react-native-meteor/pull/363 This is update fix pull from here + AsyncStorage dependency