keystone-email

keystone-email copied to clipboard

#13 - new transport / has been tested with common SES options

trentmillar

Adding AWS SES transport (with fixes)

1

This is based on the very good PR by @trentmillar in https://github.com/keystonejs/keystone-email/pull/17 I just fixed a few small issues. PS: Thanks for Keystone!

stepanh

Hi, I noticed today that the mailgun transport is missing the parameter host. At the moment I can't use it for an EU server.

BigGiantHead

Add all options to mailgun initialization

1

Solves issue #43

carlrygart

Fixed import path for NodeMailer transport

7

The relative import path for `requireOptional` was incorrect and needed an additional `../` prefixed to the existing path to allow the utility module to be located/imported

dacre-denny

Missing arguments in mailgun-js init

4

Currently only apiKey and domain are passed to mailgun-js init. In my case I need host also (using eu instances). https://github.com/keystonejs/keystone-email/blob/master/lib/transports/mailgun/index.js#L19

carlrygart

fix using nodemailer send email fail

4

{ Error: Invalid sender "admin " at SMTPConnection._formatError (/Users/feng/workspace/web/yr/node_modules/smtp-connection/lib/smtp-connection.js:528:15) at SMTPConnection._setEnvelope (/Users/feng/workspace/web/yr/node_modules/smtp-connection/lib/smtp-connection.js:723:30) at SMTPConnection.send (/Users/feng/workspace/web/yr/node_modules/smtp-connection/lib/smtp-connection.js:383:10) at SMTPTransport. (/Users/feng/workspace/web/yr/node_modules/nodemailer-smtp-transport/lib/smtp-transport.js:126:24) at /Users/feng/workspace/web/yr/node_modules/nodemailer-smtp-transport/lib/smtp-transport.js:162:21 at SMTPConnection._actionAUTHComplete (/Users/feng/workspace/web/yr/node_modules/smtp-connection/lib/smtp-connection.js:1236:5) at SMTPConnection. (/Users/feng/workspace/web/yr/node_modules/smtp-connection/lib/smtp-connection.js:319:22) at SMTPConnection._processResponse (/Users/feng/workspace/web/yr/node_modules/smtp-connection/lib/smtp-connection.js:669:16) at...

miaowing

nodemailer patch

1

seeming that #27 has been waiting 3 years to be accepted, and I needed nodemailer, here is a patch file for the latest version of master. You can use this...

mrfrase3

Corrected the module import for `nodemailerConfig` to `require('./test.config.example.js')`, to reference the file test config in the repository

dacre-denny

require ../util/requireOptional not in right path

1

HI On file lib/transports/nodemailer/index.js required path for ./util/requireOptional seems to be incorrect. It should have been ../../util/requireOptional Also one more issue on lib/util/requireOptional.js Options packages require code seems to be...

ivanclaud