localgovdrupal / localgov_geo

Entity for storing Geographic information, addresses, and areas; extensible types and plugable backends with defaults to start.
2 stars 1 forks source link

Add 'access geo browser' permissions for editors #113

Open willguv opened 7 months ago

willguv commented 7 months ago

Reported by @benhillsjones when adding a directory venue to a new Dumfries and Galloway website

He saw the following:

Image

To replicate, on a new install try adding a directory venue page with a geo location

The expected behavior is to be able to add a geo location

This was fixed locally by Rohallion by adding the 'access geo browser' permission to the Editor role

The permission needs to be added for Editor, Author and Contributor roles, as all may be required to add geo items

andybroomfield commented 7 months ago

This was added https://github.com/localgovdrupal/localgov_geo/commit/021c3af8bf4cc5285b19cf765c3c94622749e2c2 Though only fairly recently so if I site was installed before November 2023 it won't have it or any geo permissions unless explicitlly set up.

stephen-cox commented 7 months ago

An update hook to assign this could be possible, but we would need to go through every role checking if the 'access geo_entity_library entity browser pages' had been assigned and, if not, add it to the author and editor roles.

willguv commented 6 months ago

We've just made the same change locally in Essex and it resolved the issue. Would be good to address soon as I imagine every council is having to diagnose and fix this issue - it's wasted effort

@stephen-cox

MariosORION commented 4 months ago

PR https://github.com/localgovdrupal/localgov_geo/pull/121 ready for review: