Don't mess with Sidekiq::Web, include unobtrusively instead

cryo28 / sidekiq_status

Extension to Sidekiq to pass job execution metadata such as status and result back to the client

MIT License

152 stars 53 forks source link

Don't mess with Sidekiq::Web, include unobtrusively instead #4

Closed pdf closed 11 years ago

pdf commented 11 years ago

Messing with Sidekiq::Web appears unnecessary, and causes conflicts with other middleware.

pdf commented 11 years ago

Updated per comments

pdf commented 11 years ago

I cleaned up the now unnecessary change to sidekiq_status.rb, can't do anything about the whitespace change making things look ugly in web.rb though...

Sorry for the noise, was far down the rabbit whole very early this morning, have not had a whole bunch of sleep since.