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.

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

move jobs to different file, so they can be imported without triggeri… #67

Merged
merged 1 commit into from
Jul 26, 2017

Conversation

schuyler1d
Copy link
Collaborator

…ng job loop

@sreynen sreynen self-requested a review July 26, 2017 15:04
@sreynen sreynen requested a review from shakalee14 July 26, 2017 15:06
Copy link
Contributor

@shakalee14 shakalee14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@schuyler1d
Actually - do we need to also change references where job-handler.js exists?
like in package.json --> line 28 and 29

@schuyler1d schuyler1d merged commit 2d81f5d into master Jul 26, 2017
@schuyler1d schuyler1d deleted the knex-prep branch July 31, 2017 13:57
lperson pushed a commit to lperson/Spoke that referenced this pull request Sep 21, 2018
…-66-wfp_merge_main_reassign_all_matching

Revert "merging lperson/spoke: Reassign all matching (conflicts fixed)"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants