Permissions are now a mess: documentation is all about "editors, monitors, oversighters", while code is more specific like "aftv5-hide-feedback, aftv5-delete-feedback, aftv5-feature-feedback, ..."
Right now, the code largely uses "aftv5-feature-feedback" to identify "an editor" - even if the functionality has nothing to do with "featuring", which leads to confusing code (there have been multiple permissions issues before)
Either stop using "aftv5-feature-feedback" as a generic identifier for "editor" usecases (and instead start using multiple permissions like e.g. "aftv5-see-hidden-feedback" but make sure they're all assigned to the correct groups of people), or make the "editor" identifier ("aftv5-feature-feedback") less confusing.
Version: unspecified
Severity: normal