Skip to content

Remove old scanner filter and only use new filter on vuln report page [RUN ALL RSPEC] [RUN AS-IF-FOSS]

Daniel Tian requested to merge 299295-remove-old-scanner-filter into master

What does this MR do?

On the vulnerability report page, we have a dropdown to filter the vulnerability list by scanner type:

ksnip_20210517-162955

Previously, we were using a feature flag to toggle between the old version of this dropdown, which used the StandardFilter component, and a new version, which uses the ScannerFilter component. This feature flag was added so that we can test the performance of the new dropdown with the much larger dataset on production, and then be able to switch back to the old version if there were any issues.

Since we've verified that the new ScannerFilter component does not have performance issues, we no longer need the feature flag, nor the code that toggles between the old and new component. This MR removes that code and uses the new ScannerFilter code exclusively.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Daniel Tian

Merge request reports