Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Mime] rename Part/BodyFile to Part/File#48060
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Please reorder the use statements as the fabbot mentioned |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
once fabbot is ok
No markdown diff, did we forget to add some changelog entry? |
Updated! |
Uh oh!
There was an error while loading.Please reload this page.
Thank you@nicolas-grekas. |
This PR was squashed before being merged into the 6.3 branch.Discussion----------[Mailer] Fix attachment changesThe title ofsymfony/symfony#47711 did not change before merge, but actually only `attachPart` has been deprecated (renamed) in favor of `addPart`.I did not reintroduce the other methods as they may be deprecated in 7.1, so no need to document a way that may change in a few months.Also the `BodyFile` class has been renamed insymfony/symfony#48060 whereas the use in the example code was correct.Commits-------46923d0 [Mailer] Fix attachment changes
Was proposed during a chat within the core-team.
/cc@javiereguiluz