GetSimpl / amqp_mailer

Adds a new action mailer delivery method - amqp
MIT License
0 stars 0 forks source link

Make DeliveryMethod instances respond to `settings` #3

Closed nitishparkar closed 6 years ago

nitishparkar commented 6 years ago

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 rely on any settings object, added a dummy settings method which returns empty hash.