search

lardawge / carrierwave_backgrounder

Offload CarrierWave's image processing and storage to a background process using ActiveJob or Sidekiq..
736 stars 402 forks source link

Stringify sidekiq arguments #307

Closed holstvoogd closed 1 year ago

holstvoogd commented 2 years ago

Fixes Sidekiq 6.5.x warning

Sidekiq 7 will no longer allow symbol arguments

sethherr commented 1 year ago

@lardawge any chance this could get merged in and a new version could be released?

lardawge commented 1 year ago

@sethherr Can you run off master and confirm there is no breakage? I with then create a release.

Thx

sethherr commented 1 year ago

It works! No breakage for me, locally or on production.

lardawge commented 1 year ago

@holstvoogd Thanks for the PR. Merged and new version released as 0.4.3

@sethherr Thanks for the nudge. New version in the wild.