search

kshnurov / mandrill_dm

A basic Mandrill delivery method for Rails.
MIT License
46 stars 45 forks source link

add support for merge_vars #22

Closed tomasc closed 8 years ago

tomasc commented 8 years ago

see #17 

header[:merge_vars] = [ { rcpt: …, vars: [ { name: …, content: … } ] } ]

see also: https://mandrill.zendesk.com/hc/en-us/articles/205582487-How-to-Use-Merge-Tags-to-Add-Dynamic-Content

spovich commented 8 years ago

Sorry for delay. Just getting around to reviewing open PR's. Thanks for this! Just wondering if we can cleanup the reference in merge_vars, otherwise, looks good to me. @jlberglund or @kshnurov thoughts?

spovich commented 8 years ago

Hi @tomasc, @jlberglund has transferred maintenance of this gem to me. I've made a few small updates to reflect those changes. Please rebase this PR, and then it should be good to merge. Thanks!

tomasc commented 8 years ago

Hi @spovich rebased and ready, looking forward, thank you!