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

Support modern push notification payload #218

Merged
merged 3 commits into from
May 17, 2015
Merged

Support modern push notification payload #218

merged 3 commits into from
May 17, 2015

Conversation

mallowlabs
Copy link
Member

Implementation for #216

  • extend payload size to 2048 byte
  • add notification category: 'CREATE_MESSAGE'
  • add user (screen_name) to payload
  • add room_id to payload

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.51%) to 89.42% when pulling 521c9fc on mallowlabs:id/216 into 388e1ce on codefirst:master.

@suer
Copy link
Member

suer commented May 9, 2015

LGTM

mallowlabs added a commit that referenced this pull request May 17, 2015
@mallowlabs
Merge pull request #218 from mallowlabs/id/216
4dcce2e 
Support modern push notification payload
@mallowlabs mallowlabs merged commit 4dcce2e into codefirst:master May 17, 2015
@mallowlabs mallowlabs deleted the id/216 branch May 17, 2015 06:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mallowlabs @coveralls @suer