node-mandrill

node-mandrill copied to clipboard

The code is allowing the callback to be called twice: If there is no error, but the body is not a valid JSON the catch calls the callback with the...

pedroassis

No documentation in Readme for HTML

1

I had to look at a closed issue for sending html. I know it's kind of a common sense thing, but it might be nice if there was a line...

nabilfreeman

Hello! Thank you for this module! However, please consider implementing a Promise interface. Right now I have to use it like this: ``` javascript var Promise = require('bluebird'); var request...

slavafomin

At https://www.npmjs.com/package/node-mandrill the documentation says: ``` var mandrill = require('mandrill')(''); ``` and it should be `node-mandrill` as it is on current GitHub's code, right?

murilopolese

Use requestretry because sometimes recoverable errors (like `{"code":"ENOTFOUND","errno":"ENOTFOUND","syscall":"getaddrinfo"}`) can happen.

FGRibreau

Perhaps should return an error if status field in response is invalid or rejected?

1

If you try sending a mail to an invalid email address for example, mandrill will return the following response: `[{"email":"notavalidemailaddress", "status":"invalid"}]` but with status code 200, so node-mandrill thinks its...

cjroebuck