-
Notifications
You must be signed in to change notification settings - Fork 167
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
all controllers should be extending ResqueWeb::ApplicationController ins... #79
Conversation
…instead of parent application's ::ApplicationController
This pull request fixes |
👍 would love to see this merged! |
Hi I'm pretty sure this is the right thing to do, but I'm not really using Resque or ResqueWeb these days and don't really have time to test. Can anyone test this an confirm it fixes the issues like #76 and maybe others? |
If a few people can confirm this is working, I will merge it. |
all controllers should be extending ResqueWeb::ApplicationController ins...
This appears to solve a number of issues, I haven't tested it thoroughly as I'm not currently much of a resque user. However I want to help out if I can since I have commit access to this project. I'm going to merge this and release a new Gem version soon. Cheers! |
It would be nice with a new release / new Gem version! I'm probably not the only one who could benefit from this bugfix! 😄 |
@rarruda Oh sorry, I thought I did. I'll make a release today. |
...tead of parent application's ::ApplicationController