Skip to content

Refactor custom role loading

Jessie Young requested to merge jy-custom-roles-on-groups into master

What does this MR do and why?

  • Move methods outside of User class and into their own namespaced class for better separation of concerns.
  • #421265 (closed)

How to set up and validate locally

It is a refactor but if you want to test that the functionality remains the same, there are verification steps for group custom roles in the MR where they were added: !125652 (merged)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Jessie Young

Merge request reports