Skip to content

Improve standards adherence report

Jay Montal requested to merge 413718_apply_sidebar_suggestions into master

What does this MR do and why?

What

Improve standards adherence report:

  • Fix spacing issue in sidebar between status, project name, and badge
  • Update requirements section text
  • Update failure reasons text
  • Add success reasons text
  • Add learn more links per different check
  • Fix column widths

Why

Improvements based on the discussion in #413718 (comment 1561236018)

Screenshots or screen recordings

Status/Last Scanned Column Improvement

Before After
image image

Sidebar Improvements

Before After (Fail Status) After (Success Status)
image image image

How to set up and validate locally

Enable FFs

  1. In the rails console (rails c) enable the following features:
  2. adherence_report_ui (Feature.enable(:adherence_report_ui))
  3. compliance_adherence_report (Feature.enable(:compliance_adherence_report))

Setup Adherences

  1. Visit group settings and enable "Prevent approval by author" from the 'Merge Request Approval' section.
  2. This will create a couple of rows inside the project_compliance_standards_adherence table.

Visit Standard Adherence page

  1. Visit a group, under Secure (new UI) or Security & Compliance (old UI) > Compliance center
  2. Click View details to expand the sidebar component

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 #413718 (closed) & #425111 (closed)

Edited by Jay Montal

Merge request reports