Option to place payload into notification.payload.aps about node-apn HOT 3 CLOSED

I am somewhat apprehensive to put this in the library as it doesn't promote good practice, in my opinion - and the docs do suggest this - custom payload data should be in a separate namespace. I can however appreciate your need to maintain backward compatibility with existing payload structure so I will think about how to integrate this nicely for others to use as well.

from node-apn.

Why can't you just do:

var note = new apns.Notification();

note.alert = "You have a new message";
note.device = myDevice;

note.payload.aps = {};
note.payload.aps.m = "Your custom payload";

As per lines 262-264, the aps namespace only gets created if it doesn't already exist. So you can just create it as above and put something in it. The code will then generate the stuff on top if necessary.

Unless I'm mistaken and you're trying to do something different.

from node-apn.

Just had a second look. I'm not sure what I exactly missed the last time looking at this, but you're absolutely right. I just removed the fix and crafted the notification like mentioned above. Thanks a bunch!

from node-apn.