Closed ghidinelli closed 7 years ago
Thanks for the PR! Unfortunately there were some other changes merged in that have caused a conflict with this. If you're still interested, would you mind resolving them in your branch?
I merged in your changes and adapted the authenticated SMTP code to your new use_gmail strategy. Tested my authenticated account successfully. I don't have an unauthenticated SMTP gateway to test but otherwise believe it's good to go.
Only conflicting file should be the README now. Can you merge then revert the README? Or make your readme up to date @ghidinelli?
Merged, thanks!!
Thanks David - this got my 3 month old daughter her GE right before we left the country on short notice so 👍
@ghidinelli Awesome :)
Localhost delivery is unlikely to work for many people so I added support for authenticated SMTP and support over SSL/TLS if you'd like to add it in. Thanks for posting to Reddit!