Skip to content

Replace namespace_plan_check with Gitlab::saas.feature_available check

Ross Byrne requested to merge 441633-handle-dotcom-check-correctly into master

What does this MR do and why?

Resolves #441633 (closed)

Replaces the namespace_plan_check with Gitlab::saas.feature_available? check, as it's the documented way of checking.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

No visible change

How to set up and validate locally

  1. Setup to simulate SaaS and restart GDK
  2. Setup and run Customers Dot
  3. Turn on feature flag with echo "Feature.enable :trial_discover_page" | gdk rails c
  4. Register a user for a trial
  5. After getting into the app, navigate to http://gdk.test:3000/groups/<group_name>/-/discover
  6. Page should still be accessible

Related to #441633 (closed)

Merge request reports