Add dismiss functionality to GraphQL finding modal

David Pisekrequested to merge
387653-add-dismissal-functionality-to-finding-modal into master

What does this MR do and why?

  • Adds dismissal notes
  • Adds CRUD functionality

Screenshots or screen recordings

Screen_Recording_2023-03-28_at_5.31.27_pm

How to set up and validate locally

Setup

  1. Have the related FF enabled: echo "Feature.enable(:pipeline_security_dashboard_graphql)" | 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-examples/security/security-reports
  5. Run a pipeline on master

Validation

  1. Go to the pipeline's security report tab
  2. Click on a finding
  3. Dismiss the finding
  4. With the same finding:
  • add a comment
  • edit the comment
  • delete the comment

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

Edited by David Pisek

Merge request reports