Skip to content

Fix Group Import members' owner access level

What does this MR do?

This MR fixes a bug for Group Import where imported group members who had Owner access level were lowered down to Maintainer. This is due to the fact that MembersMapper was initially for Projects only (which has max access level of Maintainer) and it was later on changed to accommodate Groups. But max access level was not touched - hence all Owners of groups are lowered to Maintainers.

This change fixes that, allowing Group members to retain Owner access level.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by George Koltsov

Merge request reports

Loading