Problem/Motivation

Existing page doesn't mention Drush, which will be faster for many people.

Proposed resolution

Add commands at top of each section, including clearing cache.

CommentFileSizeAuthor
#4 2888863.patch2.44 KBjhodgdon
extend-maintenance.txt3.18 KBwturrell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wturrell’s picture

wturrell created an issue.

AsciiDoc file attached.

jhodgdon’s picture

Title: Update to Enabling & Disabling Maintenance Mode » Enabling & Disabling Maintenance Mode (extend-maintenance) should mention Drush
Status: Needs review » Needs work

Thanks for the issue! We definitely should mention the Drush commands in the topic about enable/disable of maintenance mode.
https://www.drupal.org/docs/user_guide/en/extend-maintenance.html

What we should do though is follow the style of other topics that mention Drush commands. I suggest looking at:
https://www.drupal.org/docs/user_guide/en/config-install.html

So what this patch needs:
- Site prerequisites section like in the config-install topic
- Under "Steps" the separation into using UI and using Drush, with same language as in the config-install topic. Please put the UI first. Keeping in mind this User Guide is mostly for newcomers to Drupal, they will mostly be more comfortable with the UI initially.

Thanks!

jhodgdon’s picture

Actually what I think I'd suggest under Steps, is to add the line saying:

You can use the administrative interface or Drush to enable or disable maintenance mode.

Then I would add "using the administrative interface" to the first two headers that are already there, and add a new section at the bottom called "Enabling and disabling using Drush", that would just give the commands to enable/disable maintenance mode using Drush.

Would you like to make a new version @wturrell? If I don't see a response here in a few days, we'll find someone else to do it, so either way is fine. Thanks!

jhodgdon’s picture

Status: Needs work » Needs review
Issue tags: +Translation updates needed
FileSize
2.44 KB

Here's a new patch...

eojthebrave’s picture

Status: Needs review » Reviewed & tested by the community

This looks fine to me. Thanks Jennifer.

  • jhodgdon committed aa6acab on 8.x-3.x
    Issue #2888863 by jhodgdon, wturrell, eojthebrave: Enabling...
jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks again for the issue and review! I committed this patch today, and translation group notifications are going out shortly with some other issues.

Status: Fixed » Closed (fixed)

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