search

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

Allow sidekiq_status to work with other add ons #7

Closed springbok closed 11 years ago

springbok commented 11 years ago

I've attempted to fix the problem I raised here https://github.com/cryo28/sidekiq_status/issues/6 by removing the find_template override. In my testing this seems to have fixed the problem but I would mention that I have very little idea of Sinatra & Slim so unsure if these changes are the best way to fix the problem.

cryo28 commented 11 years ago

@springbok Thanks for the pull request. Will try to look at in next weekend.

cryo28 commented 11 years ago

@springbok I removed find_template override in sidekiq web which potentially caused the problem. It is now merged to master. Please test. If problem is solved I'll release 1.0.5 in a couple of days