Skip to content

Commit da5b708

Browse files
author
Gareth Terrace
committed
Fixed sending to multiple recipients
also removed "from" from the multiple recipients documentation because you can't do that.
1 parent 5dc2aac commit da5b708

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ You can send a single message as shown above. If you have multiple messages to s
3737

3838
```js
3939
clockwork.sendSms([{ To: '447123456789', Content: 'Test!'},
40-
{ To: '447123456781', Content: 'Another one', From: 'ClockworkSMS'}], function(error, resp) {
40+
{ To: '447123456781', Content: 'Another one'}], function(error, resp) {
4141
if (error) {
4242
console.log('Something went wrong', error);
4343
} else {

lib/clockwork.js

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -97,15 +97,16 @@ var ClockworkApi = function(auth)
9797
var blankPayload = util._extend({}, rootRequest);
9898
var payload = {Message: blankPayload};
9999

100-
if (messages instanceof Array){
101-
var smsArray = [];
102-
for (var i = 0; i < messages.length; i ++) {
103-
smsArray.push({SMS:messages[i]});
104-
};
105-
payload.Message['#list'] = smsArray;
106-
} else {
100+
// GT: Bugfix to wrapper to enable multiple recipients
101+
// if (messages instanceof Array){
102+
// var smsArray = [];
103+
// for (var i = 0; i < messages.length; i ++) {
104+
// smsArray.push({SMS:messages[i]});
105+
// };
106+
// payload.Message['#list'] = smsArray;
107+
// } else {
107108
payload.Message.SMS = messages; // Should work with arrays (multiple messages) or a single object literal
108-
}
109+
// }
109110

110111
var xml = xmlbuilder.create(payload,{version: '1.0', encoding: 'UTF-8'}).end();
111112

0 commit comments

Comments
 (0)