Not actually attaching files is definitely a bug. :D


On Tue, Jun 3, 2014 at 11:53 AM, Patrick Van Stee <notifications@github.com>
wrote:

> In app/mailers/message_mailer.rb:
>
> >      from.display_name = nickname(@message.person)
> >
> >      reply_to = @conversation.mailbox_email.dup
> >
> > +    @message.attachments.each do |attachment|
> > +      attachments[File.basename(attachment.file.path)] = File.read(attachment.file.path)
> > +    end
>
> [image: :+1:] Hah. Sorry for all the trouble. Guess this is technically a
> feature not a bug.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/asm-helpful/helpful-web/pull/237/files#r13345440>.
>