mattyr / sidecloq

Recurring / Periodic / Scheduled / Cron job extension for Sidekiq
MIT License
88 stars 12 forks source link

Wrong queue shown in the UI if not defined in config #38

Open Fryguy opened 2 years ago

Fryguy commented 2 years ago

In the Sidecloq config if you don't specify the queue for a job it will properly use the queue as defined by the class. For example, the following job and worker seems to work properly in that the BatchJobMonitor job is placed on the :my_queue queue minutely.

BatchJobMonitor:
  class: BatchJobMonitor
  cron: "* * * * *" # minutely
class BatchJobMonitor
  include Sidekiq::Worker
  sidekiq_options :queue => :my_queue
  ...
end

However, there is a bug in the "Recurring Jobs" UI in that it shows default, as seen below. Instead it should be showing my_queue.

_DEVELOPMENT__Sidekiq
Fryguy commented 2 years ago

Fixed in #41