Fixed import path for NodeMailer transport

[dacre-denny]

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

[f10l]

I stumbled over the same problem. Is this module still maintained? This fix works for me.

[dacre-denny]

@fanick-ber I'm not sure if this module is maintained. Despite being mentioned in the doc's for Keystone's current release, the module itself seems to have been abandoned?

[chomamateusz]

Same issue here! How it is still unmerged?

[dacre-denny]

Not sure - it'd be nice to see this resolved!

[Findoss]

@JedWatson Hello! Please approve this PR

[joselcvarela]

For those coming from https://github.com/keystonejs/keystone/tree/master/packages/email here is a simple hack: Add this to package.json: "resolutions": { "@keystonejs/email/keystone-email": "https://github.com/joselcvarela/keystone-email" }

Then yarn add @keystonejs/email

It would be much better if this could be merged :/

[loklaan]

Another stop-gap suggestion (any largeish web project might have a couple of these):

package.json

{
  "scripts": { "postinstall": "./scripts/postinstall.sh" }
}

scripts/postinstall.sh

sed -i '' "s/'\.\.\/util/'\.\.\/\.\.\/util/" ../node_modules/keystone-email/lib/transports/nodemailer/index.js
sed -i '' "s/'pkg'/pkg/" ../node_modules/keystone-email/lib/util/requireOptional.js