Skip to content

[Mailer] docs: Add the mime type in the example #17334

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
Oct 11, 2022
Merged

[Mailer] docs: Add the mime type in the example #17334

merged 1 commit into from
Oct 11, 2022

Conversation

jorisros
Copy link
Contributor

@jorisros jorisros commented Oct 9, 2022

While embedding images the examples don't have a mimetype. Some email clients will not show a image because of missing that mimetype.

@carsonbot carsonbot added this to the 6.2 milestone Oct 9, 2022
@carsonbot carsonbot changed the title docs: Add the mime type in the example [Mailer] docs: Add the mime type in the example Oct 11, 2022
@javiereguiluz javiereguiluz modified the milestones: 6.2, 4.4 Oct 11, 2022
@jorisros @javiereguiluz
docs: Add the mime type in the example
3b08979 
While embedding images the examples don't have a mimetype. Some email clients will not work while missing that mimetype.
@javiereguiluz javiereguiluz changed the base branch from 6.2 to 4.4 October 11, 2022 15:49
@javiereguiluz
Copy link
Member

Thanks Joris.

@javiereguiluz javiereguiluz merged commit 959c06d into symfony:4.4 Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xabbuh xabbuh Awaiting requested review from xabbuh

@OskarStark OskarStark Awaiting requested review from OskarStark

Assignees
No one assigned
Labels
Mailer Status: Needs Review
Projects
None yet
Milestone
4.4
Development

Successfully merging this pull request may close these issues.
3 participants
@jorisros @javiereguiluz @carsonbot