Skip to content

Improvements to NotifyByEmail

Marek Szuba requested to merge experimental/email_runnable into master

Created by: muffato

Addresses two comments that are only 2 years old ^^:

The NotifyByEmail Runnable was really simplistic. Switching to Email::Stuffer allows adding attachments very easily, and gives a better support of multi-part emails, e.g. with HTML content, etc. Note for @james-monkeyshines : Email::Stuffer is in fact just a wrapper around Email::MIME, which you are now using in EGPipeline/RNAFeatures/EmailRNAReport.pm instead of MIME::Lite, so I expect it to work in your pipelines.

MIME::Lite is not maintained any more (http://search.cpan.org/~rjbs/MIME-Lite-3.030/lib/MIME/Lite.pm) but since it was based on Email::Sender, like Email::Stuffer is, I believe collaborators account should be able to send emails too.

Merge request reports