Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

Update PAYLOAD.md #2457

Merged
merged 2 commits into from
Jul 3, 2018
Merged

Update PAYLOAD.md #2457

merged 2 commits into from
Jul 3, 2018

Conversation

andreszs
Copy link
Contributor

@andreszs andreszs commented Jul 3, 2018

Added new section with FCM payload using the HTTP protocol.

Description

New section with FCM payload sample, including brief comments.

Related Issue

#2456

Motivation and Context

All payload data refers to the deprecated GCM service that will soon be shutdown and is mostly invalid for the FCM HTTP server protocol.

How Has This Been Tested?

Tested the payload by sending it to actual Android 4.4, 5.1 and iOS 11 phones.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

andreszs added 2 commits July 2, 2018 21:31
All the payload details from the README refer to using the deprecated GCM service. Here I've added a small example to send push notifications to the FCM legacy HTTP server protocol, including the basic fields such as title, body, icon, color, sound, and optional payload data.

Here's a sample JSON payload to send a push notification to an Android or iOS device using the FCM app server protocol:

```php
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this could be json instead of php.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@macdonst macdonst merged commit e94fb9e into phonegap:master Jul 3, 2018
@andreszs andreszs deleted the patch-1 branch July 4, 2018 14:55
dinoentersekt pushed a commit to entersekt/phonegap-plugin-push that referenced this pull request Jul 23, 2018
* Update PAYLOAD.md

All the payload details from the README refer to using the deprecated GCM service. Here I've added a small example to send push notifications to the FCM legacy HTTP server protocol, including the basic fields such as title, body, icon, color, sound, and optional payload data.

* Update PAYLOAD.md
@lock
Copy link

lock bot commented Aug 3, 2018

This thread has been automatically locked.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants