Skip to content

[Mime] Fix email rendering when having inlined parts that are not related to the content #47437

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

Merged
merged 1 commit into from
Aug 31, 2022

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Aug 30, 2022

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Refs #47279
License MIT
Doc PR n/a

@fabpot fabpot force-pushed the mime-related-issue-44 branch from ac7a142 to be97af4 Compare August 30, 2022 19:52
@fabpot fabpot merged commit 4a1d46d into symfony:4.4 Aug 31, 2022
@fabpot fabpot deleted the mime-related-issue-44 branch August 31, 2022 08:00
xabbuh added a commit to xabbuh/symfony that referenced this pull request Sep 11, 2022
This method was introduced in symfony#47034, but its usage was removed again when
merging symfony#47437 up into the 6.2 branch.
fabpot added a commit that referenced this pull request Sep 13, 2022
This PR was merged into the 6.2 branch.

Discussion
----------

[Mime] remove unused getDisposition() method

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

This method was introduced in #47034, but its usage was removed again when merging #47437 up into the 6.2 branch.

Commits
-------

746cf35 remove unused getDisposition() method
glengemann pushed a commit to glengemann/symfony that referenced this pull request Sep 15, 2022
This method was introduced in symfony#47034, but its usage was removed again when
merging symfony#47437 up into the 6.2 branch.
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.

3 participants