Ionic-native: Push Plugin: Add `clearAllNotifications` method and return promises/observables

[FdezRomero]

Hey there!

I started using the Push plugin and noticed that is missing the newer clearAllNotifications method.

I also noticed that the ionic-native wrapper is using the plugin's success and error callbacks instead of returning a promise/observable as it happens with most of the other plugins, which is confusing considering the docs say plugins return either of these.

Any chance to add this missing method and use promises/observables instead of callbacks? I'm guessing there are people already using this plugin so it should keep supporting the callbacks and return a promise/observable if none is specified.

Thanks!

[ihadeed]

Hey,

Thanks for pointing it out.

We will have to rewrite the Push plugin using the @CordovaInstance wrapper in order to achieve that functionality. I guess we can add that to the road map (no set date, though).

[FdezRomero]

Thanks @ihadeed! Looking forward to it as many apps use this plugin.

[FdezRomero]

Hi @ihadeed!

Any news on adding the missing push.clearAllNotifications() method to the Push plugin?

Thanks!

[FdezRomero]

Thanks @ihadeed! :+1: