Skip to content

License check with FF for work in progress limits

Scott Stern requested to merge ss/fix-frontend-ff-on-wip-limits into master

What does this MR do?

Wip limits license was not being checked when pushing the FF. Wip Limits is a starter only feature and needs to be guarded as so.

Repro:

  • Feature.enable(:wip_limits)
  • make sure you have a license
  • confirm you can do the following

2020-01-15_13.27.40

  • Remove your license && rebuild your gdk instance.
  • make sure you cannot see the gear.

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 Scott Stern

Merge request reports