localgovdrupal / localgov_alert_banner

Alert banner module for LocalGovDrupal
GNU General Public License v2.0
4 stars 8 forks source link

fix/1.x 363 emergency publisher perms #364

Closed andybroomfield closed 1 month ago

andybroomfield commented 1 month ago

Fix #363

What does this change?

Adds the following permisions to the emergency publisher role

This is to allow users with just the emergency publisher role to access the manage alert banners screen from the toolbar menu.

How to test

Create a user with just the emergency publisher role and check they can navigate to content -> alert banners in the top menu.

How can we measure success?

Users with this role can now access the alert banners screen on default setup.

Have we considered potential risks?

Gives access to access content overview, which they should'nt really need. Risk is small since typically those with this role would have access to that anyway. However since this PR is about those with just this role, it does give rise to asking if a seperate menu item should now be provided.

Images

n/a

Accessibility

n/a

andybroomfield commented 1 month ago

So this was due to changes to emergency publisher installing permissions that are not defined during the test runs.