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

Added android support to cordova-gimbal-bridge #2

Merged
merged 36 commits into from
Oct 7, 2016

Conversation

koga73
Copy link
Contributor

@koga73 koga73 commented Oct 3, 2016

Added support for Android.
Lifted GimbalAdapter from the UA-extensions repo
Added GimbalAdapterService for persistence when the app is in background / closed
Created a JavaScript API for both platforms with the ability to start / stop monitoring
Updated documentation

koga73 and others added 25 commits September 19, 2016 17:26
@rlepinski
Copy link
Contributor

Looks pretty good. Some of the files have inconsistent spacing (tabs vs spaces) and code style (if ( vs if(). We need to get it fixed before we merge. Let me know if you need some help with that.

@koga73
Copy link
Contributor Author

koga73 commented Oct 5, 2016

Tweaked the code a bit and updated formatting for better consistency

@koga73
Copy link
Contributor Author

koga73 commented Oct 7, 2016

I believe this is ready to be merged now

@rlepinski
Copy link
Contributor

@koga73 We are working on a few more changes for the cordova-urbanairship plugin. After we release it ill merge and release this PR. Thanks for all the work!

@rlepinski rlepinski merged commit d0a4a86 into urbanairship:master Oct 7, 2016
@rlepinski
Copy link
Contributor

@koga73 Everything is released, Thanks again for your help!

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.

None yet

3 participants