Fix schedule hooks when enqueuing configured job #792
Fix schedule hooks when enqueuing configured job #792PatrickTulskie merged 2 commits intoresque:masterfrom
Conversation
https://github.com/resque/resque-scheduler#hooks enqueue_at works as documented, but schedules enqueued from a config have not been calling the before/after schedule hooks. fixes resque#679
cryptomail
left a comment
There was a problem hiding this comment.
Hey there :)
@PatrickTulskie
As if I have any authoritative say in approvals..... haha :) 👿
BUT.... I would like to take this opportunity to thank you for your work on this project.
I was wondering if we could level set expectations on integration of this work, because monkey patching our way out of this is very unsightly and doesn't feel that great :)
Please let me know how I can help.
Again, thanks for your time and attention.
Hey sorry for the looooong delay. Just got back from getting married and I'm trying to get out from under my work backlog. This is high on my list though, so please just bare with me. |
Rebased the changes from #681 to update / run CI, closes #679