Skip to content
This repository has been archived by the owner on Feb 20, 2019. It is now read-only.

Update the background jobs configuration information #3002

Merged
merged 2 commits into from
Apr 23, 2017
Merged

Update the background jobs configuration information #3002

merged 2 commits into from
Apr 23, 2017

Conversation

settermjd
Copy link
Contributor

@settermjd settermjd commented Apr 18, 2017 •
edited
Loading

This PR:

  • Updates the documentation about and the correct background job configuration to use.

Relates To

#2988

@settermjd settermjd self-assigned this Apr 18, 2017
@mmattel
Copy link
Contributor

mmattel commented Apr 18, 2017

👍

phil-davis
phil-davis reviewed Apr 18, 2017
View reviewed changes
admin_manual/configuration_server/background_jobs_configuration.rst Outdated
The disadvantage of this mechanism, when compared to the `Webcron`_ service, is that it requires regular visits to the page for it to be triggered.

.. note:: Especially when using the Activity App or external storages, where new
files are added, updated, or deleted one of the two methods below should be
Copy link
Contributor

Choose a reason for hiding this comment

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

now can s/below/above
or say "one of the other methods should be..."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @phil-davis. I've made a follow-up commit to correct this oversight. Always great to know you're there, keeping me honest.

@settermjd settermjd merged commit 0482fb0 into master Apr 23, 2017
@settermjd settermjd deleted the update-background-job-configuration branch April 23, 2017 11:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@phil-davis phil-davis phil-davis left review comments

@PVince81 PVince81 Awaiting requested review from PVince81

@jakobsack jakobsack Awaiting requested review from jakobsack

Assignees

@settermjd settermjd

Labels
Automation WIP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@settermjd @mmattel @phil-davis