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

updated example to work with RN 0.20 #14

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

Conversation

manask88
Copy link
Collaborator

ive updated the example so that it works with RN 0.20. Im planning to send another PR that will add support for Android after this one is merged in

updated rn demo

updated rn demo

updated rn demo

updated rn demo

updated rn demo
@pwmckenna
Copy link
Owner

wow, this is awesome! sorry for the delay. will add you as contributor to both npm and github repos so you can get this stuff out faster. thanks!

@pwmckenna
Copy link
Owner

also, i'm pretty out of the RN scene at the moment, so my review probably doesn't mean much at this point. feel free to merge if you feel good about it.

@pwmckenna
Copy link
Owner

what's your npm username?

@manask88
Copy link
Collaborator Author

same as github, manask88

@pwmckenna
Copy link
Owner

k, added your as a contributor so you should be able to publish as well. thanks again!

@dsibiski
Copy link

@manask88 Is there any progress with this? I noticed that you mentioned Android support...do you need help getting it added in?

@zxcpoiu
Copy link

zxcpoiu commented Apr 28, 2016

for android, you can look into https://github.com/kprimice/react-native-sensor-manager
since both repo's api are similar and licensed in ISC, we can do the porting job with the two as I mentioned at #10

@pwmckenna
Copy link
Owner

@manask88 looks like @connected-jallen added android support in a pr if you want to take a look. @dsibiski @zxcpoiu hopefully that helps you out as well.

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.

4 participants