Skip to content

No beta label for CS on SM in 16.7

Aleksei Lipniagov requested to merge 435270-sm-remove-cs-beta-label-16-7 into master

What does this MR do and why?

Do what this MR - !139965 (merged) - does, but updates the correct view that is shown to 16.7 users.
In the original MR, the updated view is behind the FF until 16.8 - so for 16.7 the original MR is no-op which is not the intent, it seems.

This is a new requirement for Cloud Connector. What we agreed on previously is discussed in that issue description, plus this edit from Katie (which simplifies things a bit and avoids some additional logic). This MR changes the original intent.

Please involve Legal/Design or other interested parties if needed. As with any last-minute change, it may be risky if not communicated well.

@tmccaslin executive team requested this change and legal approved it.

Note that if will not merge this MR into 16.7, there is no need to use it, because the entire view affected by this MR will be dropped in %16.8: !139141 (merged)

Screenshots or screen recordings

Before After
before Screenshot_2023-12-18_at_13.14.41

How to set up and validate locally

  • Pull this branch.
  • Ensure the FF is disabled on your instance:
    • no action needed unless you tested something Cloud Connector-related before)
    • in rails console, Feature.disable(:updated_ai_powered_features_menu_for_sm)
  • Check the http://localhost:3000/admin/application_settings/general
  • No Beta label should appear near the CS checkbox (check the screenshot)
  • On master, you will still see the Beta label

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 #435270 (closed)

Edited by Taylor McCaslin

Merge request reports

Loading