Skip to content

Mounts CI variables setting form correctly

Payton Burdette requested to merge pb-fix-broken-admin-variables into master

What does this MR do?

Fixes an admin bug.

The bug is the element ID is not handled correctly to mount the ci variables component. Resulting in no component being mounted in the admin view.

This MR fixes this.

See bug reported here: #326806 (closed)

Screenshots (strongly suggested)

Admin Mount

Screen_Shot_2021-04-02_at_10.31.41_AM

Group Mount

Screen_Shot_2021-04-02_at_10.32.01_AM

Project Mount

Screen_Shot_2021-04-02_at_10.32.17_AM

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 Payton Burdette

Merge request reports