Skip to content

Remove "Try now" CTA from code suggestions non owner alert

Ross Byrne requested to merge 435295-remove-cta-from-cs-alert into master

What does this MR do and why?

Resolves https://gitlab.com/gitlab-org/gitlab/-/issues/435295.

Remove "Try now" CTA from code suggestions non-owner alert.

Based on !139787 (comment 1694575686) the CTA link has been updated to https://docs.gitlab.com/ee/user/project/repository/code_suggestions/saas.html#use-code-suggestions.

Screenshots or screen recordings

Before After
Screenshot_2023-12-14_at_11.39.35 Screenshot_2023-12-14_at_11.39.17

How to set up and validate locally

Setup feature flags

Feature.enable(:code_suggestions_ga_non_owner_alert)
Feature.enable(:ai_assist_ui)
Feature.enable(:ai_assist_flag)
  1. Setup GDK to simulate SaaS.
  2. Login as Admin and set a public group use EE plan that is not trial with instructions point 3 here
  3. On the group's admin page: http://gdk.test:3000/admin/groups/<group_name>, pick a non-owner group member to impersonate
  4. View the group and projects in it to interact and observe the alert.

To clear user callout to show alert again, run ::Users::Callout.last.destroy

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #435295

Edited by Ross Byrne

Merge request reports