Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expo SDK 36: RNCNetInfo.getCurrentState got 3 arguments, expected 2 #700

Closed
srmagura opened this issue Jan 7, 2020 · 3 comments
Closed

Comments

@srmagura
Copy link

srmagura commented Jan 7, 2020

I had Bugsnag working in my app on Expo SDK 35, but it is no longer working after upgrading to SDK 36.

I have upgraded to @bugsnag/expo and @bugsnag/js to 6.5.0 to get the @react-native-community/netinfo fix discussed in #668.

I receive this error immediately after starting my app:

(I can only test on Android right now, but can try to reproduce on iOS later this week if desired.)

The error goes away if I comment out all references to bugsnag.

expo diagnostics output and package.json dependencies here: https://pastebin.com/KHmvKF15

@srmagura
Copy link
Author

srmagura commented Jan 8, 2020

@bengourley
Copy link
Contributor

Thanks for the report and the detailed steps to reproduce ✨

We have a fix for this (#691) which is not yet released. I will check it against your repro and get it shipped it ASAP.

@bengourley
Copy link
Contributor

Verified that this fixes the problem on your repro (had to comment out the font loading as the assets were not included).

The fix has now been released in v6.5.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants