Skip to content

Remove Compliance framework setting at sub-group

What does this MR do and why?

For sub groups ensure the can edit variable is a boolean for editing and adding compliance framework. It was being evaluated as string and not turning off compliance framework functionality in sub-groups

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After
General-settings--Settings--Gitlab-Org-_-GitLab General-settings--Settings--Test-group-_-GitLab

How to set up and validate locally

https://docs.gitlab.com/ee/user/group/compliance_frameworks.html

  1. Navigate to a top level group and go to Settings -> Compliance Frameworks
  2. Create a compliance framework
  3. Ensure at top level group you can edit and make it default
  4. Create a sub group and go to its Settings -> Compliance Frameworks
  5. Ensure that the top level frameworks show
  6. Ensure that you can not edit the frameworks or make default

MR acceptance checklist

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

Related to #416153 (closed)

Edited by Nate Rosandich

Merge request reports