- 🇧🇪Belgium siemen_hermans
Created a new tag for the view based on the previous patch and the existing administerusersbyrole_edit_access tag logic.
- Status changed to Needs review
over 1 year ago 3:19pm 10 July 2023 - last update
over 1 year ago 1 pass - 🇩🇪Germany Anybody Porta Westfalica
Would it perhaps be possible to provide this as MR and write a test? That way can be proved it works as expected for now and in the future.
- Assigned to adamps
- 🇬🇧United Kingdom adamps
Sorry for the delay I will review this soon. Either a patch or a MR is fine by me, so I'll review the patch. Of course the community would appreciate tests, however this module is accepting bug fixes without insisting on them.
- last update
7 months ago 1 pass - 🇬🇧United Kingdom adamps
Sorry for the delay
Here is an updated patch
- name the tag administerusersbyrole_access tag and check all the permissions (view and edit) as described in the IS.
- use a function for the common code rather than duplicating it
Please can someone review/test?
- Issue was unassigned.