Once the module is setup and ready, a drush command should be provided that will allow enabling/disabling "run on cron" for system monitor. This will make it easier for developers to manage this setting without requiring the use of the UI.

This command should be compatible with drush 8 and 9.

Additionally, this ticket should include updating the associated drush command documentation here:
https://www.drupal.org/docs/8/modules/system-monitor/drush

The update should include a sample usage of the command and what it does.

CommentFileSizeAuthor
#9 interdiff-3074973-6-9.txt1.61 KBrobpowell
#9 system-monitor-3074973-9.patch1.6 KBrobpowell
#6 system-monitor-3074973-6.patch1.62 KBrobpowell

Comments

merauluka created an issue. See original summary.

merauluka’s picture

merauluka commented
Assigned: Unassigned » robpowell

Assigning to Rob Powell.

robpowell’s picture

robpowell
he/him
Location Boston
commented

@merauluka, is this a config value that already exists? If we are planning on it being a config value, do we need a custom command or will drush cset suffice?

merauluka’s picture

merauluka commented

It will be a config value. But I was hoping that we could provide it as a shorter command than cset. Because cset would require typing in the full name of the config file and the setting to set.

This would be, basically, a shortcut.

merauluka’s picture

merauluka commented
Issue summary: View changes

Updated the original ticket description to include updating the documentation page as part of this ticket.

robpowell’s picture

robpowell
he/him
Location Boston
commented
StatusFileSize
new system-monitor-3074973-6.patch1.62 KB

Create a drush 8 command to turn cron on and off. I am holding off on updating documentation until we approval the naming convention. I also created a follow up issue to create drush 9 commands, 3094757.

robpowell’s picture

robpowell
he/him
Location Boston
commented
Status: Active » Needs review
merauluka’s picture

merauluka commented

@robpowell After looking through the drush commands, I think that if we can create a third function and then pass the on/off value through to it it would prevent code duplication. What do you think?

robpowell’s picture

robpowell
he/him
Location Boston
commented
StatusFileSize
new system-monitor-3074973-9.patch1.6 KB
new interdiff-3074973-6-9.txt1.61 KB

Let's give this a shot.

  • merauluka committed 874faaf on 8.x-1.x authored by robpowell 
    Issue #3074973 by robpowell: Implement drush command to enable/disable...
merauluka’s picture

merauluka commented
Status: Needs review » Fixed

Committed! Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.