Skip to content

Conversation

@xabbuh
Copy link
Member

@xabbuh xabbuh commented Aug 3, 2020

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets
License MIT
Doc PR

Before:

Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException: You have requested a non-existent service "mailer.logger_message_listener".

After:

A client must have Mailer enabled to make email assertions. Did you forget to require symfony/mailer?

@fabpot
Copy link
Member

fabpot commented Aug 6, 2020

Thank you @xabbuh.

@fabpot fabpot merged commit d81eb08 into symfony:4.4 Aug 6, 2020
@xabbuh xabbuh deleted the pr-32930 branch August 6, 2020 06:13
fabpot added a commit that referenced this pull request Aug 21, 2020
…logger listener (xabbuh)

This PR was merged into the 5.2-dev branch.

Discussion
----------

[FrameworkBundle] properly choose the best mailer message logger listener

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

Properly merge the changes from #37729 and #37728.

Commits
-------

6cffc79 properly choose the best mailer message logger listener
This was referenced Aug 31, 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