search

chalettu / pushwoosh-nodejs

This is a node module I created to help consume Pushwoosh services in NodeJS
11 stars 5 forks source link

send duplicate messages #1

Closed salexch closed 9 years ago

salexch commented 9 years ago

in this.sendMessage line #50: var sendOptions = defaultConfig;

not cloning the defaultConfig I fixed it with underscorejs

to

var sendOptions = _.extend({}, defaultConfig);

this fixed the bug.

thank you for sharing the module!!

oriefrati commented 9 years ago

Hi, i think i have same issue, you problem was that the plugin was sending new push message and also the old one again? something like:

Send Msg 1 Receive Msg 1

Send Msg 2 Receive Msg 1 Receive Msg 2

Send Msg 3 Receive Msg 1 Receive Msg 2 Receive Msg 3

salexch commented 9 years ago

It was a long time ago.. sounds like a problem I had

oriefrati commented 9 years ago

OK i will try your fix.

Thanks!.