Gate trial CTA's using redirect based on FF

David Hamp-Gonsalvesrequested to merge
cdot-16943-redirect-sm-auto-subscription into master

What does this MR do and why?

Previous MR - !232188 (merged) switched out urls that used new_self_managed_trials_path but that misses the cases that use the non-sm path but are also rendered in SM cases.

This change reverts the CTA link changes in favor of redirecting at the route layer to cover all navigation sites(based on the same FF)

References

Screenshots or screen recordings

No visual changes

How to set up and validate locally

How to set up and validate locally

  1. Restart GDK in self-managed mode: export GITLAB_SIMULATE_SAAS=0 and gdk restart
  2. Log in as administrator and visit http://127.0.0.1:3000/dashboard/projects
  3. Click Start free trial — should redirect to the Customers Portal trial form (FF is off by default)
  4. Enable in_instance_self_managed_trial_activation at http://127.0.0.1:3000/rails/features/
  5. Click Start free trial again — should render the in-app form at /-/trials/new

MR acceptance checklist

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

Edited by David Hamp-Gonsalves

Merge request reports