This task is to track removing current references to the Status Page from the current 8.x-1.x branch.

Work is already in progress to split the Status Page feature into a submodule as part of #3087448: Implement status check logic for Status Page.

This will allow the module to run in a "headless" mode as needed.

CommentFileSizeAuthor
#3 system_monitor--3094580--status-page-remove--3.patch6.91 KBmerauluka
#2 system_monitor--3094580--status-page-remove--2.patch6.31 KBmerauluka

Comments

merauluka created an issue. See original summary.

merauluka’s picture

merauluka commented
Status: Active » Needs review
StatusFileSize
new system_monitor--3094580--status-page-remove--2.patch6.31 KB

I have prepared a patch with the changes necessary to pull the status page logic out of the greater system monitor module.

Gitlab branch
https://git.drupalcode.org/project/system_monitor/compare/8.x-1.x...task...

merauluka’s picture

merauluka commented
StatusFileSize
new system_monitor--3094580--status-page-remove--3.patch6.91 KB

Missed removing a route from the routing file. Attaching an updated patch.

robpowell’s picture

robpowell
he/him
Location Boston
commented

This is looking good to me and blocking some other work so I'll go ahead and merge it in. It would be nice to link to where the status page is going to be ie sub module or stand alone.

robpowell’s picture

robpowell
he/him
Location Boston
commented
Status: Needs review » Reviewed & tested by the community

  • merauluka authored 59e7add on 8.x-1.x 
    Issue #3094580 by merauluka, robpowell: Split Status Page into a...
merauluka’s picture

merauluka commented
Status: Reviewed & tested by the community » Fixed
merauluka’s picture

merauluka commented

Giving Rob Powell credit for the review on this ticket.

Status: Fixed » Closed (fixed)

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