Use Resque.validate instead of custom .validate_job! #181

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@bogdan

Job class validation is now a part of resque API.
No need for custom method anymore.

It has slightly different behavior. That is why there is a need to change test. But it still fits problem we are trying to solve.

@bogdan bogdan Use Resque.validate instead of custom .validate_job!
Job class validation is now a part of resque API.
No need for custom method anymore
28d3cf0
@meatballhat

@bogdan I know it's been a long time, but any chance you can resolve the conflict(s) and re-push? I'd love to get this in 😸

@bogdan

@meatballhat I think you can simply fix it yourself in master. My glory won't get hurt.

@bogdan bogdan closed this Oct 15, 2013
@meatballhat meatballhat added a commit to meatballhat/resque-scheduler that referenced this pull request Oct 16, 2013
@meatballhat meatballhat Use Resque.validate instead of custom .validate_job!
Job class validation is now a part of resque API.
No need for custom method anymore

Adapted from #181 submitted by @bogdan
c195da2
@bjones bjones pushed a commit to bjones/resque-scheduler that referenced this pull request May 30, 2014
@meatballhat meatballhat Use Resque.validate instead of custom .validate_job!
Job class validation is now a part of resque API.
No need for custom method anymore

Adapted from #181 submitted by @bogdan
799de19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment