[TIMOB-2424] Push notifications not passing full payload to app

GitHub Issue	n/a
Type	Bug
Priority	Low
Status	Closed
Resolution	Fixed
Resolution Date	2011-04-17T01:59:04.000+0000
Affected Version/s	n/a
Fix Version/s	Release 1.6.0 M01
Components	iOS
Labels	ios, iphone, notifications, push, release-1.6.0, rplist
Reporter	James Wragg
Assignee	Reggie Seagraves
Created	2011-04-15T03:19:21.000+0000
Updated	2011-04-17T01:59:04.000+0000

Description

Extra data passed in a Push Notification is stripped before it get's to the app.
e.g. acme1 & acme2 are not readable:

{

"aps" : {
    "alert" : "You got your emails.",
    "badge" : 9,
    "sound" : "bingbong.aiff"
},
"acme1" : "bar",
"acme2" : 42

}

Comments

Ralf Pfeiffer 2011-04-15

This sticket has been sitting unassigned, with no milestone and low priority...

I guess this came in directly, outside of HelpDesk.

Assigning to myself for triage
Ralf Pfeiffer 2011-04-15

Blain please take a look at whether we allow this.

Ralf Pfeiffer 2011-04-15

For developers they can use this workaround:

{

"aps" : {

   "alert" : "You got your emails.",
   "badge" : 9,
   "sound" : "bingbong.aiff",
   "payload" : { 
        "acme1" : "bar",
        "acme2" : 42
   }

} }

We will be looking in to this for the next release following R1.5.0

James Wragg 2011-04-15
I raised the ticket after being told by help desk this was an enhancement (I disagree):
http://developer.appcelerator.com/helpdesk/view/40681">http://developer.appcelerator.com/helpdesk/view/40681

For reference, the example was pulled from this page:
http://developer.apple.com/library/ios/#documentation/NetworkingInternet/Conceptual/RemoteNotificationsPG/ApplePushService/ApplePushService.html"> http://developer.apple.com/library/ios/#documentation/NetworkingInt...

This bug is causing my push notifications to have the very ugly work-around of having essential payload data in the user-visible message & parsing it out in my app, e.g.

"aps" : {
```
   "alert" : "There are new matches to your search (a834,1,624632)",
   "badge" : 9,
   "sound" : "bingbong.aiff"
   
```
},

As you can imagine, users are confused.
Ralf Pfeiffer 2011-04-15

What about my example above of the payload workaround? I don't see the last code example by James Wragg using that.

We will look at the the format of the first example and from HelpDesk and evaluate whether we think we should support that going forward.
Jeff Haynie 2011-04-15

(from [4d2377fb38804d1177ac8369181daa276f688609]) [#2424] Include full dictionary in push notifications, but still copy APS props to toplevel for backcompat. https://github.com/appcelerator/titanium_mobile/commit/4d2377fb38804d1177ac8369181daa276f688609"> https://github.com/appcelerator/titanium_mobile/commit/4d2377fb3880...
Stephen Tramer 2011-04-15

Forgot to tag fixed-in-qa.
Don Thorp 2011-04-15

Correcting the milestone, this was committed on master. It is not in the 1.5.1 release.
Thomas Huelbert 2011-04-15

1.6.0.0db09d1e 4gt touch, 3.1.3 iphone

JSON Source