Hi,

Just to clarify a bit more right now the documentation just to get started with Unified Push notifications and cordova is a page long and includes much code that you should include by hand. When we create our own specific version of the PushPlugin we can reduce this to about the first 3 lines of the documentation. It will still work with the command line tools and with the build, but it will be specific to AeroGear Unified push so contributing this code back to the original plugin will not be very helpful. 

If we fork, then we own it :-)

Right, I'm fine with that


Ideally, it would continue to work with things like Phonegap Build and other tools the Phonegap/Cordova community bring forth.  Does the current plugin benefit from the mainstream Push-as-a-Service vendors?  I am not sure if they participate in the plugin's future feature enhancements - can we keep up or outpace them? 

Don't really know what you mean when you say that Push-as-a-Service vendors will participate in the plugin future enhancements. This plugin is very basic 'native' push integration into cordova. It's only 370 lines of java code and about the same on iOS.


On Sep 27, 2013, at 8:26 AM, Lucas Holmquist wrote:

+1
On Sep 27, 2013, at 7:36 AM, Kris Borchers <kris@redhat.com> wrote:

Seems like a good idea to me.

On Sep 27, 2013, at 3:11 AM, Erik Jan de Wit <edewit@redhat.com> wrote:

Hi,

We have already got some documentation how to use cordova with unified push by using the PushPlugin. But I was thinking about forking it and making our own version I think Matthias had this idea as well. If you look at the PushPlugin now there is a lot of things that are platform specific. For instance to register you should do:

if (device.platform == 'android' || device.platform == 'Android') {
  pushNotification.register(successHandler, errorHandler, {"senderID":"replace_with_sender_id", "ecb":"onNotificationGCM"});
} else {
  pushNotification.register(tokenHandler, errorHandler, {"badge":"true", "sound":"true", "alert":"true", "ecb":"onNotificationAPN"});
}

Then there are some more platform specific thing that we could remove as well, because we know that you are going to use this together with Unified Push we could make the process of registering a lot easier and create only one register function that works for both platforms:

pushNotification.register(successHandler, errorHandler, {"badge":"true", "sound":"true", "alert":"true", "senderID":"123", "ecb":"onNotification"});

Making it easier for people to get started with cordova and for us less documentation to maintain ;)

_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev

_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev

_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev

_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev