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

upgrade examples so that they are compatible with latest react-native version #195

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

Conversation

liusy182
Copy link

Examples were still based on older versions of react-native which is not compatible with latest versions. This PR upgrades examples so that they are compatible with current version of react-native.

@Kureev
Copy link
Owner

Kureev commented Oct 31, 2016

Hi @liusy182!

Thanks for your contribution! Have you checked if it still works after this PR?

@liusy182
Copy link
Author

Hi @Kureev, I have checked all examples on iOS and they all worked. I did not check android though.

@Kureev
Copy link
Owner

Kureev commented Nov 1, 2016

Would be nice if somebody who has some time (can I ask you, @satya164?) try Android examples first before I merge it in.

@satya164
Copy link

satya164 commented Nov 1, 2016

I'll test that tonight. BTW what do you think about migrating the example to exponent? It'll be super -easy for people to try it out then. And we don't have any native code in the repo which we need to update on RN updates.

Should be easy to convert to exponent using exp convert command.

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.

3 participants