Meteor-Community-Packages / raix-push

DEPRECATED: Push notifications for cordova (ios, android) browser (Chrome, Safari, Firefox)
https://atmospherejs.com/raix/push
MIT License
514 stars 197 forks source link

Add FCM Firebase Cloud Messaging support #354

Closed fishsaidno closed 4 months ago

fishsaidno commented 5 years ago

This is still a WIP, opening this PR for code review, suggestions and and testing. With my testing so far, it seems pretty solid, however I have not yet tested in a production-like environment.

Have not yet implemented webpush (desktop notifications) using FCM, this PR only focuses on cordova integration.

Documentation still needs a lot of work, docs/FCM.md is a quick overview of the changes. Most of that should probably be edited into the existing documentation.

Will update this further next week.

Cheers!

raix commented 5 years ago

@FishSaidNo looks good so far - just remove comments / unused code - ping me if you want me to review

shivang007 commented 5 years ago

@FishSaidNo any updates on this?