Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ability to specify global queue for ActionMailer extension #2242

Closed
wants to merge 1 commit into from

Conversation

jjb
Copy link
Contributor

@jjb jjb commented Mar 19, 2015

Threw this together just to get feedback.

If you like the idea of this feature I will write tests and improve the code.

(clearly default_options will not be named that since it's generic to ActionMailer after being mixed in, and also I'll do something about the repeated code)

@mperham
Copy link
Collaborator

mperham commented Mar 19, 2015

It's easy to reopen the class in your initializer and use sidekiq_options. WDYT?

@jjb
Copy link
Contributor Author

jjb commented Mar 19, 2015

Okay, sounds good.

For anyone coming across this in the future wanting the same thing, here's how:

# config/initializers/sidekiq.rb
module Sidekiq
  module Extensions
    class DelayedMailer
      sidekiq_options queue: "mailer"
    end
  end
end

@jjb jjb closed this Mar 19, 2015
@jjb jjb deleted the configurable-mailer-queue branch March 19, 2015 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants