Skip to content

Use vulnerability-mr field for security modal

What does this MR do and why?

Instead of using the MR information directly from the finding, this change switches to using the vulnerability-mr field.

The goal of the change is to keep the querying consistent with other parts of the application.

More context: #404894 (comment 1372558834)

Screenshots or screen recordings

Screenshot_2023-05-16_at_3.37.00_pm

How to set up and validate locally

Setup

  1. Have the related FF enabled: echo "Feature.enable(:standalone_finding_modal)" | rails c
  2. You'll need an EE License
  3. You'll need to have runners enabled (See $2408961 for setting up a runner)
  4. Import https://gitlab.com/gitlab-org/govern/demos/sandbox/minac/test-remediations
  5. Run a pipeline on master

Validation

  1. Go to the pipeline's security report tab
  2. Create MR via the "Resolve with merge request" button
  3. Go back to pipeline's security tab and click on same finding
  4. Verify that the MR note is showing and the "Resolve with merge request" button is not visible

MR acceptance checklist

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

Edited by David Pisek

Merge request reports

Loading