Skip to content

Minor permissions refactoring

Chris Oelmueller requested to merge refactor/permissions into master

What does this MR do?

  • make some permission checks more sequential (i.e. easier to follow)
  • extract permission handling from WorkGroupControl into new WorkGroupPermissions
  • 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

Merge request reports