search

evendis / mandrill-rails

Webhook processing and event decoration to make using Mandrill with Rails just that much easier
MIT License
288 stars 36 forks source link

Update webhook processor #30

Closed estepnv closed 9 years ago

estepnv commented 9 years ago

Add sync webhook handling

tardate commented 9 years ago

@estepan thanks for this. That sync hook definition is untidily non-standard...whatever! Merging now..

estepnv commented 9 years ago

@tardate thank you! It's a great feature this gem really missed.

tardate commented 9 years ago

@estepan np! btw, if possible, it would be great if you could contribute some (real) "sample" payloads - just send them in another P/R if you can (redacted as you wish).

The existing files in spec/fixtures/webhook_examples/sync_blacklist.json,sync_whitelist.json are ones I just "made up" based on the API docs. But shouldn't really assume the API docs are 100% correct.

estepnv commented 9 years ago

@tardate np. As soon as I get this feature tested in my staging env.