Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix before/after_schedule hooks not called when enqueued with config #681

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix before/after_schedule hooks not called when enqueued with config #681

wants to merge 4 commits into from

Commits on Jul 8, 2019

  1. move config to a shared method

    @jweir
    jweir committed Jul 8, 2019
    Configuration menu
    Copy the full SHA
    d059a01 View commit details
    Browse the repository at this point in the history

  2. enqueue(config) calls {before, after}_schedule methods 

    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
    @jweir
    jweir committed Jul 8, 2019
    Configuration menu
    Copy the full SHA
    ce39a0b View commit details
    Browse the repository at this point in the history

Commits on Jul 9, 2019

  1. Merge branch 'master' into bugfix/schedule_hooks_not_called

    @jeremy
    jeremy committed Jul 9, 2019
    Configuration menu
    Copy the full SHA
    b979733 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into bugfix/schedule_hooks_not_called

    @jweir
    jweir committed Jul 9, 2019
    Configuration menu
    Copy the full SHA
    7f62292 View commit details
    Browse the repository at this point in the history