Skip to content

[Mailer] Use addPart instead of embed* or attach*. #17372

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
javiereguiluz merged 1 commit into from
Oct 20, 2022
Merged

[Mailer] Use addPart instead of embed* or attach*. #17372

javiereguiluz merged 1 commit into from
Oct 20, 2022

Conversation

@hiddewie
Copy link
Contributor

@hiddewie hiddewie commented Oct 19, 2022 •
edited
Loading

Fixes #17303
Ref #17353
Ref symfony/symfony#47711
Ref symfony/symfony#47462

This PR contains the documentation for simplifications for adding parts to Emails.

@carsonbot carsonbot added this to the 6.2 milestone Oct 19, 2022
@hiddewie hiddewie mentioned this pull request Oct 19, 2022
Closed
mdoutreluingne
mdoutreluingne reviewed Oct 19, 2022
View reviewed changes
@carsonbot carsonbot changed the title Use addPart instead of embed* or attach*. [Mailer] Use addPart instead of embed* or attach*. Oct 20, 2022
@javiereguiluz javiereguiluz merged commit eca3116 into symfony:6.2 Oct 20, 2022
@javiereguiluz
Copy link
Member

javiereguiluz commented Oct 20, 2022

Hidde, thanks for taking the time to update the mailer docs to use the new methods!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@mdoutreluingne mdoutreluingne mdoutreluingne left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Mailer Status: Reviewed

Projects

None yet

Milestone

6.2

Development

Successfully merging this pull request may close these issues.

[Mime] Simplify adding Parts to an Email

4 participants

@hiddewie @javiereguiluz @mdoutreluingne @carsonbot