Skip to content

Add alert banner with breaking changes for security polices

Artur Fedorov requested to merge 444511-policies-new-changes-banner into master

What does this MR do and why?

Breaking changes needs to be communicated to the user.

This issue adds info alert banner for security policies list

  1. We should have only one alert at a time; this one will take priority over others.
  2. After dismiss, don't show again

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

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Description Video
All alert banners visible by prio banner.mov

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Enable feature flag:

Feature.enable(:security_policies_breaking_changes)
  1. Go to Group or project
  2. Secure -policies
  3. Make sure alert with breaking changes for 17.10 is in place
  4. Dismiss it
  5. Make sure approval policy new name banner is visible
  6. Dismiss it too
  7. Refresh page make sure, banners are gone
  8. Go to Dev tools Application -> Local Storage -> Remove keys security_policies_breaking_changes and security_policies_scan_result_name_change
  9. Refresh page, make sure banners are visible

Related to #444511 (closed)

Edited by Artur Fedorov

Merge request reports