Skip to content

Use shared_min_access_level parameter

Jerry Seto requested to merge access-dropdown-api-access-level into master

What does this MR do and why?

Use shared_min_access_level parameter

The min_access_level parameter is not used on the project groups endpoint 🤦. The correct parameter to use is shared_min_access_level in order to filter groups properly

How to set up and validate locally

  1. Create a project
  2. Invite a group to the project with the developer role
  3. Go to repository settings (Settings -> Repository)
  4. See that the Allowed to create dropdown for creating and editing protected branches contains the group
  5. See that the Allowed to merge and Allowed to push and merge dropdowns for creating and editing protected branches contain the group
  6. Update the max role of the invited group to Reporter (Manage -> Members -> Groups tab)
  7. See that the dropdowns do not contain the group

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 Jerry Seto

Merge request reports