-
-
Notifications
You must be signed in to change notification settings - Fork 221
first version support nette mailer for send tracy mail #299
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
Conversation
|
Thanks! |
9cfa3aa to
e3a251b
Compare
|
@dg Should not be the mailer optional? What if you don't have it in your app? |
|
You must enable it via option 'netteMailer' in config. |
| ->setClass('Tracy\ILogger') | ||
| ->setFactory('Tracy\Debugger::getLogger'); | ||
| ->setFactory('Tracy\Debugger::getLogger') | ||
| ->addSetup('setMailer', [$builder->getDefinition('mail.mailer')]); |
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.
@dg Where is the option netteMailer? :-)
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.
@f3l1x oh, ofcourse, in tracy extension missing setting to other mail system...
|
I am maybe stupid, but |
|
I did not merge this PR as it was. Merged was 48118da |
|
I see. That's it ;-) Thank you. |
|
Nice! 👍 |
prepare to mailer support for send