Addresses https://pagure.io/fedora-infrastructure/issue/6222
rebased
I'm not sure this comment is correct, it seems more inline with "If the person doing the action is the former main admin, keep them admins"
That can be simplified to retain_access = retain_access.lower() in ['true', '1'] :)
retain_access = retain_access.lower() in ['true', '1']
@pingou thanks for the review. Fixed it.
These four lines can be combined into one I think: retain_access = str(retain_access).lower() in ['true', '1']
retain_access = str(retain_access).lower() in ['true', '1']
Tests are passing, thanks for addressing this :)
Pull-Request has been merged by pingou
Addresses https://pagure.io/fedora-infrastructure/issue/6222