Upgrading redux-offline and netinfo dependencies for react native

[Ashish-Nanda]

Issue #, if available:

Description of changes:
Upgrading redux-offline and netinfo dependencies for react native to use latest versions. This is because newer versions of NetInfo deprecated some APIs that were used by the version of redux-offline we were using.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[ucallQA]

is there an ETA when this will be merged?

[DrWorkhard]

Waiting for this as well, if I understand correctly this makes amplify compatible with expo 37

[Ashish-Nanda]

@DrWorkhard we were waiting for @react-native-community/async-storage to be supported in expo because redux-offline has a dependency on it, and and this change would not work in Expo without it.

This just happened over the weekend: react-native-async-storage/async-storage#368 (comment)

We will probably test the change in both bare and expo managed apps and then merge the PR later this week.

[DrWorkhard]

@DrWorkhard we were waiting for @react-native-community/async-storage to be supported in expo because redux-offline uses has a dependency on it, and and this change would not work in Expo without it.

This just happened over the weekend: react-native-community/async-storage#368 (comment)

We will probably test the change in both bare and expo managed apps and then merge the PR later this week.

Thank you so much, good to hear and keep up the great work!

[duongvanba]

There are any way to apply this pull request for my projet?

[theharness]

@Ashish-Nanda Any updates on timeframe? Looks like Expo SDK 38 just released with react-native-community/async-storage included.

[jvgaeta]

Any update on when this will be merged? Thanks!

[Ashish-Nanda]

Hi,
Thank you for your patience. We realized the PR needs some more changes and have a bit more testing to do before we merge and release. We have prioritized this now, and aim to get it out before end of next week.

[amhinson]

Tested apps generated with RN CLI & Expo with Verdaccio, and all seems to be working as expected 👍

[Ashish-Nanda]

Tested apps generated with RN CLI & Expo with Verdaccio, and all seems to be working as expected 👍

Thanks for verifying @amhinson!

[amhinson]

LGTM 🚢

[elorzafe]

Thanks @Ashish-Nanda ! 👍

[mikeRChambers610]

I don't understand, what is the fix for the issue? NPM install new aws packages? Which ones?

[theharness]

@mikeRChambers610 Just install the latest aws-app sync package and you’ll have the fix.