- Issue created by @sahilgidwani
- Issue was unassigned.
- Status changed to Needs review
7 months ago 4:59am 17 June 2024 - 🇮🇳India sahilgidwani Jaipur
I have committed proposed changes and created a merge request.
- Status changed to Needs work
7 months ago 5:53am 18 June 2024 - 🇬🇧United Kingdom scott_euser
Thanks for raising. However we should not give create, update, and delete access. Only view access should be allowed. So admin permission attribute would need to be removed and replaced with an access handler so existing permission can be returned for all but view.
Thanks!
- 🇬🇧United Kingdom scott_euser
I would also be okay if you wanted to create a seperate permission for configuring like 'administer site settings types' but this would then also need an update hook to maintain the status quo for existing installs.
-
scott_euser →
committed d71fa737 on 2.0.x authored by
sahilgidwani →
Issue #3454970 by sahilgidwani, scott_euser: Change admin_permission for...
-
scott_euser →
committed d71fa737 on 2.0.x authored by
sahilgidwani →
- Status changed to Fixed
5 months ago 10:52am 24 July 2024 Automatically closed - issue fixed for 2 weeks with no activity.