Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update monolog_email.rst #13107

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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Update monolog_email.rst #13107

wants to merge 1 commit into from

Conversation

abdounikarim
Copy link
Contributor

Hello,

I just add a link 馃槈

Hello,

I just add a link 馃槈
@@ -7,7 +7,7 @@ How to Configure Monolog to Email Errors
.. caution::

This feature is not compatible yet with the new :doc:`Symfony mailer </mailer>`,
so it requires using SwiftMailer.
so it requires using :doc:`SwiftMailer </email>`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We at Symfony want the new Mailer is used, so adding a link and make it more simple to go to SwiftMailer does not makes sense to me.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@OskarStark actually, monolog isn't compatible with Mailer. That's why i add the link 馃槈

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the PR has been merged recently
symfony/symfony#33456
so IMO ne more need to promote swiftmailer email logging

exept for older version maybe :)

@HeahDude
Copy link
Contributor

Closing here for the given reasons. Thanks

@HeahDude HeahDude closed this Feb 19, 2020
@HeahDude HeahDude mentioned this pull request Feb 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.