Skip to content

Puts Vue LFS badge behind a feature flag

Phil Hughes requested to merge ph/moreVueFileListChecks into master

What does this MR do?

This puts the LFS badge entity in the GraphQL query behind a feature flag. We saw some errors in production around this LFS badge so it would be good to test this part behind a feature flag without disabling the whole feature.

Also this adds an extra check around the path locks button to make sure that no event listeners get added to a button that doesn't exist.

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

Merge request reports