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

Moved postp, daily, backlog search threads start/stop to app.js. #5975

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

p0psicles
Copy link
Contributor

@p0psicles p0psicles commented Jan 9, 2019

  • PR is based on the DEVELOP branch
  • Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review
  • Read the contribution guide

Implements: #5558
Fixes: Enabling / Disabling automated postprocessor or changing the frequency needs a restart to take effect.

Added config - search options to disable the backlog and daily search schedulers.
Used the MedusaApp object in app.py to create setters, so we can change the schedulers without requiring a restart.

@ghost
Copy link

ghost commented Jan 9, 2019

DeepCode analyzed this pull request.
There are no new issues.

@ghost
Copy link

ghost commented Jan 19, 2019

DeepCode analyzed this pull request.
There is 1 new info report.

Click to see more details.

@pymedusa pymedusa deleted a comment Jan 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant