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

NetInfo Fix #347

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

systemlevel
Copy link

No description provided.

@ricaragao
Copy link

This is needed to RN >=0.60

@ricaragao
Copy link

Maybe we can check RN version and choose the import, what do you think?

@AndriiUhryn
Copy link

@ricaragao When this is going to be live? Mine project is in production and due to the latest Xcode changes I am in need to migrate to the newest RN. I stuck now because of NetInfo issue.

@KyrosDigital
Copy link

Any update on when this will be ready?

@ricaragao @hugocourthias, is this going to be live soon?
As far as I can tell this is the only update needed to the library in order to upgrade our react native versions to >=0.60

Happy to help test this as well if needed.
We have quite a few react native apps in production and in development, we love this package and are happy to help contribute to it if needed.

@ricaragao
Copy link

@KyrosDigital , I needed to create a fork, because I couldn't wait for Pull request revision.

My fork is https://github.com/ricaragao/react-native-meteor , it works fine to my RN 0.61

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

Successfully merging this pull request may close these issues.

5 participants