search

bradpauly / griddler-mailgun

Mailgun adapter for the griddler gem.
MIT License
17 stars 27 forks source link

update from field to reply-to if exists, added message-url into messa… #12

Closed ziweizhou closed 5 years ago

ziweizhou commented 8 years ago

sometimes email has a reply to field, which is lost after the normalization. I replaced to field with the value from reply-to. also the message-url field is lost as well after the normalization. I included it in the headers.

please let me know if anything else I need to change.

bradpauly commented 8 years ago

Whoops, I meant to make that comment here.

ziweizhou commented 8 years ago

indeed, however i dont see anywhere else to pass these informations except header.

btw, i also added the file attachment parse logic as well. I believe the current code disgard them

Regards.

strzibny commented 7 years ago

@bradpauly I am now at a point where I need this 'url' param as well. Mailgun docs are confusing but it is included with store() and notify route. Having direct access to what Mailgun sent would be helpful as well.

bradpauly commented 5 years ago

@ziweizhou @strzibny sorry this has been open for so long. If you're still interested in passing these along, vendor_specific is the best place to do it. See https://github.com/bradpauly/griddler-mailgun/pull/22