Skip to content

Put new call behind feature flag

What does this MR do?

A new call that was introduced in this MR was not put behind FF. This MR addresses that.

Screenshots (strongly suggested)

  1. Created a new project, made it public, and applied the protected branch settings as mentioned here:

Screenshot_from_2020-12-09_16-44-56

  1. With another user, I forked the project, updated a file, created a MR targetting the original project's master branch. Trying to merge:

    Screenshot_from_2020-12-09_16-43-10

    • With this MR fix:

    Screenshot_from_2020-12-09_16-44-19

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

Related to #292450 (closed)

Edited by Etienne Baqué

Merge request reports

Loading