Skip to content

Omit sub-groups from trial eligibility

What does this MR do?

Adds a condition to the Namespace.eligible_for_trial scope so that only top-level namespaces are returned. Previously, we returned all namespaces, including sub-groups, belonging to the current user, but we only allow trials on top-level namespaces.

Screenshots

With the following configuration of groups:

image

This MR takes the trial group selection menu from Before to After:

Before After
image image

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

Addresses #228781

Edited by Dallas Reedy

Merge request reports