Skip to content

[Mime] Simplify adding Parts to an Email #17303

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

Closed
fabpot opened this issue Sep 28, 2022 · 2 comments · Fixed by #17372
Closed

[Mime] Simplify adding Parts to an Email #17303

fabpot opened this issue Sep 28, 2022 · 2 comments · Fixed by #17372
Labels
actionable Clear and specific issues ready for anyone to take them. hasPR A Pull Request has already been submitted for this issue. Mime
Milestone

Comments

@fabpot
Copy link
Member

fabpot commented Sep 28, 2022

Q A
Feature PR symfony/symfony#47462
PR author(s) @fabpot
Merged in 6.2
@fabpot fabpot added this to the 6.2 milestone Sep 28, 2022
@carsonbot carsonbot added the Mime label Sep 28, 2022
@wouterj
Copy link
Member

wouterj commented Oct 18, 2022

This should be documented together with #17353

The sections that require update (using addPath() instead of attach*()/embed*()) are https://symfony.com/doc/current/mailer.html#file-attachments and https://symfony.com/doc/current/mailer.html#embedding-images I think.

@wouterj wouterj added actionable Clear and specific issues ready for anyone to take them. help wanted Issues and PRs which are looking for volunteers to complete them. labels Oct 18, 2022
@hiddewie
Copy link
Contributor

I made #17372 to document both symfony PRs.

@wouterj wouterj added hasPR A Pull Request has already been submitted for this issue. and removed help wanted Issues and PRs which are looking for volunteers to complete them. labels Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
actionable Clear and specific issues ready for anyone to take them. hasPR A Pull Request has already been submitted for this issue. Mime
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants