Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mailer setup problem #20

Closed
markbiria opened this issue Oct 5, 2018 · 1 comment
Closed

mailer setup problem #20

markbiria opened this issue Oct 5, 2018 · 1 comment

Comments

@markbiria
Copy link

There is just a simple mistake in sample code :
in blog.go line 103
ab.Config.Core.Mailer = defaults.LogMailer{}

in blog.go line 117
defaults.SetCore(&ab.Config, *flagAPI, false)

it overrides the mailer with its default ...
i mean if u change line 103 to use your mailer implementation in my case mailgun

ab.Config.Core.Mailer = mailgunmailer

it will be overriden to default mailer that is std.out logger ....

Thanks 👍

@aarondl
Copy link
Member

aarondl commented Oct 10, 2018

Thanks for pointing this out. I removed the offending line because it made a very useless mailer. :)

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

No branches or pull requests

2 participants