drewr / postal

Clojure email support
MIT License
586 stars 85 forks source link

Allow file-name and description to be set on all content parts #67

Closed bo-chen closed 9 years ago

bo-chen commented 9 years ago

This allows file-name to be set on all parts of the message, not just attachments. Here's an example of including some csv data in an email:

(send-email
  {:from "bo@"
   :to addresses
   :subject "hi"
   :body [{:type "text/plain"
           :content "I attached a csv"}
          {:type "text/csv"
           :file-name "data.csv"
           :content csv-content}]})
charles-dyfis-net commented 9 years ago

Could you add test coverage?

bo-chen commented 9 years ago

Added

drewr commented 9 years ago

Thanks @bo-chen! Added a way to keep the dotos in there in 8a2de00. Mind merging it in? git fetch && git merge --no-ff origin/thread-conditionals.

bo-chen commented 9 years ago

Merged the contribution and removed clojure 1.3 and 1.4 dependency as discussed.

drewr commented 9 years ago

Looks good :+1:

I've invited you to collaborate, so you can merge when ready!

bo-chen commented 9 years ago

Thanks it's in!