Skip to content
This repository has been archived by the owner on Aug 20, 2021. It is now read-only.

Display attached images in HTML emails #32

Merged
merged 2 commits into from
Jun 14, 2014
Merged

Display attached images in HTML emails #32

merged 2 commits into from
Jun 14, 2014

Conversation

omarev
Copy link
Contributor

@omarev omarev commented Jun 13, 2014

Enables embedding of images inside html template
<img src="cid:logo.png" alt="Logo">

Similar header is added for every attachment and can be used as in above example
Content-ID:<logo.png>

Enables embedding of images inside html template
<img src="cid:logo.png" alt="Logo">

Similar header is added for every attachment and can be used as in above example
Content-ID:<logo.png>
@acelaya
Copy link
Collaborator

acelaya commented Jun 13, 2014

Nice feature!
Let me check it out, but seems very simple and useful

@acelaya
Copy link
Collaborator

acelaya commented Jun 13, 2014

You are on time to fix it. Just send more commits to your branch.
You could do something like

$basename = basename($attachment);
$part->id = $basename;
$part->filename = $basename;

@acelaya
Copy link
Collaborator

acelaya commented Jun 13, 2014

Just checked with the last modification and works great! I didn't knew how this worked.
As soon as you update that I'll merge it.

Now the embedding of images should work correctly.
@omarev
Copy link
Contributor Author

omarev commented Jun 14, 2014

As you suggested I've added the omitted basename function from earlier.

acelaya added a commit that referenced this pull request Jun 14, 2014
Display attached images in HTML emails
@acelaya acelaya merged commit ebf6b0b into acmailer:master Jun 14, 2014
@omarev omarev deleted the patch-1 branch June 14, 2014 21:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants