Minor permissions refactoring
What does this MR do?
- make some permission checks more sequential (i.e. easier to follow)
- extract permission handling from
WorkGroupControl
into newWorkGroupPermissions
- a few type hints
How confident are you it won't break things if deployed?
Should be okay, code was mostly just moved around
Links to related issues
How to test
- configure workgroups for the various application modes
With different users:
- contact / apply
- view workgroup list
Checklist
-
no test: nothing new -
no unrelated changes -
asked someone for a code review -
set a "for:" label to indicate who will be affected by this change -
use "state:" labels to track this MR's state until it was beta tested -
added an entry to CHANGELOG.md -
no release notes / beta testing needed
Edited by Chris Oelmueller