Skip to content

Add compare_accessibility_report to MergeRequest

Max Orefice requested to merge mo-add-compare-accessibility-report-to-mr into master

Part of #211389 (closed)

This feature will be behind a feature flag called accessibility_report_view.

What does this MR do?

This MR adds a new method called compare_accessibility_reports which will be used in a follow up MR to generate an accessibility report diff on a MR widget.

Standard approach to leverage our generic service comparison to create a diff. The implementation is similar to our test_report feature: compare_test_report.

Why are we doing this?

Step #11 to introduce Accessibility Report comparison with our Merge Request Widget.

The entire feature has been developed on an MVC branch, this MR aims to break it down in smaller chunk.

Screenshots

You can see how the feature will look like with the screenshot provided below:

image

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team

Dependencies

Merge request reports