Skip to content

Refactor vulnerability report cluster filter

Samantha Ming requested to merge 376340-cluster-filter into master

What does this MR do and why?

This MR refactors the cluster filter to detangle from the simple_filter.vue and instead use the new more basic querystring_sync.vue component. The goal will make each of the filter components more flexible for customization. Please read this epic for more information > &8872 (closed).

Describe in detail what your merge request does and why.

Screenshots or screen recordings

image

How to set up and validate locally

  1. Enable the FF echo "Feature.enable(:refactor_vulnerability_filters)" | rails c
  2. Security & Compliance > Vulnerability report
  3. The dropdown should function as before

Note: if you don't see any cluster vulnerabilities, you can mock it using Tweak

For other methods, please see this comment > sming-gitlab/my-gitlab#52 (comment 1182622809)

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

Edited by Samantha Ming

Merge request reports

Loading