search

msavin / SteveJobs

A simple jobs queue that just works (for Meteor.js)
Other
207 stars 35 forks source link

Add Result for reschedule call #79

Closed gerwinbrunner closed 4 years ago

gerwinbrunner commented 5 years ago

https://github.com/msavin/SteveJobs/blob/27cdf83a256a6c38a357cc442892c03955612b1f/package/server/imports/actions/reschedule/index.js#L16-L20

I think it would be good to have also the result field passed to the reschedule call and then pushed that to the history.

msavin commented 5 years ago

@gerwinbrunner good idea, can you PR it?

msavin commented 4 years ago

@gerwinbrunner on second thought, I think using this.set and this.get makes more sense. Seeing as how there has been low engagement on this topic, I will close it.

If you have a use case for it, would love to know