Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

Nodemailer transport doesn't found #14

Open
ivanclaud opened this issue Dec 6, 2016 · 6 comments
Open

Nodemailer transport doesn't found #14

ivanclaud opened this issue Dec 6, 2016 · 6 comments

Comments

@ivanclaud
Copy link

Hi,

I'm trying to send email using nodemailer but nodemailer package doesn't found if i install using npm install keystone-email

So done a workaround, installed from github npm install https://github.com/keystonejs/keystone-email.git

@LorbusChris
Copy link
Contributor

You need to manually add nodemailer to your project as well.
If it is not yet listed in your package.json, run:

npm install --save nodemailer

@ivanclaud
Copy link
Author

ivanclaud commented Dec 7, 2016

Thanks @LorbusChris

Yes I had installed nodemailer initially .

But the issue is , If I set transport as nodemailer then it's explicitly looking for lib/transports/nodemailer/index.js which is not found

@LorbusChris
Copy link
Contributor

LorbusChris commented Dec 7, 2016

Oh yea, you're actually on to something. There hasn't been published a new version to npm since addition of the nodemailer transport. For the time being, in your dependencies in package.json you can specify

"keystone-email": "https://github.com/keystonejs/keystone-email.git",

to pull master from this repo directly on npm install.

@JedWatson do you mind publishing a new version to npm? I'd be excited to see how the nodemailer transport performs out in the open =)

@neagle
Copy link

neagle commented Dec 8, 2016

I just ran into this issue as well. Thanks to @ivanclaud for pointing it out. Happy to switch to listing the Github repo for now, but it's be great to get the npm version updated. 😄

@ivanclaud
Copy link
Author

Just publishing package to npm may not work . Because I ran into this (#15) as well

@Brmm
Copy link

Brmm commented Jun 9, 2017

Is there a reason this is not published on npm yet? Linking to a github master branch is not wise in production.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants