Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Mailer] Fix body renderer check #48366

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 28, 2022
Merged

[Mailer] Fix body renderer check #48366

merged 1 commit into from
Nov 28, 2022

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Nov 28, 2022

Q A
Branch? 6.2
Bug fix? yes
New feature? no
Deprecations? no
Tickets n/a
License MIT
Doc PR n/a

What I did in #47049 does not work as one can have many message listeners. So, we need to do this check after all listeners are run.

@carsonbot carsonbot added this to the 6.2 milestone Nov 28, 2022
@fabpot fabpot force-pushed the mailer-body-renderer-fix branch 2 times, most recently from ee2446a to 73a7417 Compare November 28, 2022 16:56
@fabpot fabpot force-pushed the mailer-body-renderer-fix branch from 73a7417 to 8bb7522 Compare November 28, 2022 17:20
@fabpot fabpot merged commit 8b8e1d4 into symfony:6.2 Nov 28, 2022
@fabpot fabpot deleted the mailer-body-renderer-fix branch November 28, 2022 17:57
@fabpot fabpot mentioned this pull request Nov 28, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicolas-grekas nicolas-grekas nicolas-grekas approved these changes

Assignees
No one assigned
Labels
Bug Mailer Status: Reviewed
Projects
None yet
Milestone
6.2
Development

Successfully merging this pull request may close these issues.
3 participants
@fabpot @nicolas-grekas @carsonbot