Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#4 | 2888863.patch | 2.44 KB | jhodgdon |
extend-maintenance.txt | 3.18 KB | wturrell |
Comments
Comment #1
wturrell CreditAttribution: wturrell as a volunteer commentedwturrell created an issue.
AsciiDoc file attached.
Comment #2
jhodgdonThanks 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!
Comment #3
jhodgdonActually 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!
Comment #4
jhodgdonHere's a new patch...
Comment #5
eojthebraveThis looks fine to me. Thanks Jennifer.
Comment #7
jhodgdonThanks again for the issue and review! I committed this patch today, and translation group notifications are going out shortly with some other issues.