-
Notifications
You must be signed in to change notification settings - Fork 20
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
Changed and added (a lot of...) Android functionality #6
base: master
Are you sure you want to change the base?
Conversation
Gonzalo, why don't you merge this? |
Im reviewing this right now and testing, I will keep you posted throughout the day. Thanks @akreienbring for your contribution. |
Any update on this @gonzaloaune ? Seems like a lot of good stuff here. Edit: Just noticed "His plugin has iOS support, but I think I broke the compatibility with my changes.", which may be the reason why the merging is taking a while? |
Yes, I'm actually updating the iOS side since this pull request broke that functionality. I expect to have it ready by later today. |
Any update regarding this? |
@gonzaloaune any update on your review? will be using your plugin if you get this merged :) |
I'm also looking forward to this getting merged. |
I moved to this plugin... |
Hi Gonzalo, thanks for the ground work you made.
I was in need of a plugin that uses the new GCM implementation. E.g. subscribing to topics was not possible with the (deprecated) phongegap-push-plugin.
So I found yours and reworked the Android part. It now has much more functionality and mimics the behaviour of phongegap-push-plugin in most of its parts.
Unfortunatly I don't think that the iOS implementation is still working because I changed your callback philosophy to an event based approach.
I hope you will merge this request and adjust the iOS code to my proposals.
Greetings,
André