fall back to sendmail if smtp server is disabled #136
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the comment, LGTM.
@@ -0,0 +1,11 @@ | |||
const nodemailer = require('nodemailer'); | |||
|
|||
const sendmail = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer if it was part of _config.yml
as emailFallback
or something like that.
I Considered that but I didn't see that as a fallback but as an alternative |
But it is a fallback. It would be an alternative if you had an option to use one or another. But here the sendmail is only used if SMTP fails. |
Why is there yarn update's in this? Seem's totally unrelated to the change |
I had some weird dependency issue I solved with an upgrade. Can remove it if yiu want |
we dont want it as an alternative, we only want it for when smtp fails |
What is the point of reviews if you ignore them. |
i didnt ignore it had a green button |
No description provided.