I have modified the code to only show the status message for non admin pages.
It would be great if the status message was actually a block instead, this would give site builders the choice on what pages the block should appear on.

CommentFileSizeAuthor
#2 2908494-do-not-show-admin-pages.patch855 bytesjozzy_a

Comments

jozzy_a created an issue. See original summary.

jozzy_a’s picture

jozzy_a commented
StatusFileSize
new 2908494-do-not-show-admin-pages.patch855 bytes
nsciacca’s picture

nsciacca
She/Her/Hers
Primary language English
commented

+1 for this patch. I also think allowing it to be exposed as a block would be beneficial so the position & visibility rules could be configured.

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented

Hey, @jozzy_a tested your patch, and it's working perfectly.Good job!
Definitely, agree that in the feature we should implement expose as a block and extend this module to be more dynamic...But we should create a new issue for this feature request.

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Status: Needs review » Reviewed & tested by the community

  • jozzy_a authored 0c12bfc on 8.x-1.x 
    Issue #2908494 by jozzy_a: Patch to only show status message for non...
labboy0276’s picture

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

I took this one step further. I created a checkbox on the alert form that asks if you want to show on admin pages or not. I still gave jozzy_a credit.

  • labboy0276 authored d8879ae on 7.x-1.x 
    Issue #2908494 by labboy0276: Patch to only show status message for non...

Status: Fixed » Closed (fixed)

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