Skip to content
This repository has been archived by the owner on Nov 1, 2021. It is now read-only.

Update Notification.php #8

Merged
merged 1 commit into from
Jul 3, 2017
Merged

Conversation

sanderjongsma
Copy link
Contributor

Based on OneSignal API documentation, included_segments is not required.

JSON example:

{
  "app_id": "5eb5a37e-b458-11e3-ac11-000c2940e62c",
  "include_player_ids": ["6392d91a-b206-4b7b-a620-cd68e32c3a76","76ece62b-bcfe-468c-8a78-839aeaa8c5fa","8e0f21fa-9a5a-4ae7-a9a6-ca1f24294b86"],
  "data": {"foo": "bar"},
  "contents": {"en": "English Message"}
}

Based on OneSignal API documentation, `included_segments` is not required. 

JSON example:
```
{
  "app_id": "5eb5a37e-b458-11e3-ac11-000c2940e62c",
  "include_player_ids": ["6392d91a-b206-4b7b-a620-cd68e32c3a76","76ece62b-bcfe-468c-8a78-839aeaa8c5fa","8e0f21fa-9a5a-4ae7-a9a6-ca1f24294b86"],
  "data": {"foo": "bar"},
  "contents": {"en": "English Message"}
}
```
@namnv609
Copy link
Owner

namnv609 commented Jul 3, 2017

Thank you for you contributing. I'll release it soon as soon possible 😃!

@namnv609 namnv609 merged commit 7ccc14b into namnv609:master Jul 3, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants