We need to create a way for users to configure the status page with the following options:

  • Set status page title
  • Set roles that are able to view status page
  • Set the path for the status page
  • Make the status page active/inactive
  • Set the monitors that are accessible to the status page
CommentFileSizeAuthor
#5 system_monitor--status-page-setup--3087447--5.patch30.67 KBmerauluka
#4 system_monitor--status-page-structure--3087447--4.patch29.53 KBmerauluka
#2 system_monitor--status-page-structure--3087447--2.patch29.46 KBmerauluka

Comments

merauluka created an issue. See original summary.

merauluka’s picture

merauluka commented
StatusFileSize
new system_monitor--status-page-structure--3087447--2.patch29.46 KB

Included in patch

  • Added status page config entity settings
  • Added status page access check logic
  • Added permissions and role checking on status page entity
  • Added dynamic route generation for status pages using the path set on the status page

Diff on Gitlab

https://git.drupalcode.org/project/system_monitor/compare/8.x-1.x...task%2Fissue-3087447--status-page-management-form

merauluka’s picture

merauluka commented
Status: Active » Needs review
merauluka’s picture

merauluka commented
StatusFileSize
new system_monitor--status-page-structure--3087447--4.patch29.53 KB

Attaching a new patch that resolves an issue with the Status Page delete form.

merauluka’s picture

merauluka commented
StatusFileSize
new system_monitor--status-page-setup--3087447--5.patch30.67 KB

Another updated patch. This one includes updates for the menu links that were overlapping and causing unexpected behavior for the end user.

robpowell’s picture

robpowell
he/him
Location Boston
commented

Looks good.

robpowell’s picture

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

  • merauluka committed 6eb0f15 on 8.x-1.x 
    Issue #3087447 by merauluka, robpowell: Create status page management...
merauluka’s picture

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

Sweet. Thanks @robpowell!

Committed to the repo.

  • merauluka committed 6eb0f15 on 9.x-1.x 
    Issue #3087447 by merauluka, robpowell: Create status page management...

Status: Fixed » Closed (fixed)

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