Skip to content

Add mid-term banner

Angelo Gulina requested to merge ag/332478-mid-term-banner into master

What does this MR do?

Relates to gitlab-org/gitlab#332478, this is a chain of 2 MRs that add a banner to the Subscription Breakdown view in order to let customers with an active subscription of type license file to activate their subscription with an activation code.

Add SubscriptionActivationBanner 👈 you are here
Plug SubscriptionActivationBanner in and make it dismissible !64618 (merged)

In this MR we add the new component and tests, preparing the ground for later plugging it in, and making it dismissible.

A feature flag is not used because this feature is for self-managed instances. The feature is behind an Application Setting and it's not released nor documented. To test it locally, see the following steps.

Local Testing

To see the button

  1. You should have a (legacy) license in your instance. You can refer to this for how to do that.

  2. Make sure you have set the following environment vars (usually in your terminal) and restart gdk

    • GITLAB_LICENSE_MODE is set to test
    • CUSTOMER_PORTAL_URL is set to https://customers.stg.gitlab.com
  3. Use Rails console to run ApplicationSetting.current.update(cloud_license_enabled: true)

  4. Visit Admin > Subscription

  5. To see the banner you can use this patch: banner.patch

Screenshots (strongly suggested)

Only for showcase purpose.

desktop

Screen_Recording_2021-06-23_at_19.35.21 Screenshot_2021-06-23_at_19.35.14

mobile

Screenshot_2021-06-23_at_19.35.42

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • 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 Angelo Gulina

Merge request reports