Skip to content

Make instances of DeliveryMethod respond to `settings`

Choose a tag to compare

View all tags
@nitishparkar nitishparkar released this 15 Mar 08:19
· 41 commits to master since this release
v0.2.1
86ed8e4
This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode. 
Make DeliveryMethod instances respond to `settings` (#3)

Mail::Message expects DeliveryMethod to have a `settings` instance
method. Without it, we were getting `NoMethodError`, and sidekiq was
triggering the same job repeatedly. Since AmqpMailer's DeliveryMethod
doesn't need settings, added a dummy settings method
Assets 2
Loading