Change to allow multiple attachments. #1

Merged
merged 3 commits into from Nov 11, 2011

Conversation

Projects
None yet
2 participants
Contributor

gollyg commented Nov 10, 2011

Hey,

Don't know if you are interested, but I had a Drupal 7 requirement to send multiple attachments. I used your code as a base and then made some mods. Feel free to pull them back in if they are of any use.

Cheers
James

Owner

ngmaloney commented Nov 10, 2011

Thanks for the code. Ill post an update on the site.

Best,

NM
On Nov 10, 2011 1:04 AM, "gollyg" <
reply@reply.github.com>
wrote:

Hey,

Don't know if you are interested, but I had a Drupal 7 requirement to send
multiple attachments. I used your code as a base and then made some mods.
Feel free to pull them back in if they are of any use.

Cheers
James

You can merge this Pull Request by running:

git pull https://github.com/gollyg/Attachment-Email multiple_attachments

Or you can view, comment on it, or merge it online at:

#1

-- Commit Summary --

  • Change to allow multiple attachments.

-- File Changes --

M attachment_email.php (139)

-- Patch Links --

https://github.com/ngmaloney/Attachment-Email/pull/1.patch
https://github.com/ngmaloney/Attachment-Email/pull/1.diff


Reply to this email directly or view it on GitHub:
#1

@ngmaloney ngmaloney added a commit that referenced this pull request Nov 11, 2011

@ngmaloney ngmaloney Merge pull request #1 from gollyg/multiple_attachments
Change to allow multiple attachments.
6673314

@ngmaloney ngmaloney merged commit 6673314 into ngmaloney:master Nov 11, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment