Skip to content

Conversation

@alex-friedl
Copy link
Collaborator

@alex-friedl alex-friedl commented Feb 1, 2017

When sending notifications to Android applications that are built with Cordova and the PhoneGap PushPlugin, the background notification handler in the app is not called.

According to the documentation of the Push Plugin (https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#android-behaviour), they recommend sending only a data object as payload.

In this PR I added a flag that can be used to send GCM messages in the way recommended for the PhoneGap Push plugin

@appfeel appfeel merged commit 65b1fed into appfeel:master Feb 8, 2017
@appfeel
Copy link
Owner

appfeel commented Feb 8, 2017

Really great! Thanks!

@appfeel
Copy link
Owner

appfeel commented Feb 8, 2017

@alex-friedl there is a branch from your pull request that is not covered with tests. I would like to publish this new feature to npm, could you provide a test for the remaining branch?

I've integrated with coveralls, so now you can run npm run test-cover and will see report of uncovered branch.

Thanks again for your effort! :)

@alex-friedl
Copy link
Collaborator Author

Hello @appfeel ,
my test setup was actually incorrect, hence my branches weren't actually tested.

I added a new PR #31

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.

2 participants