Skip to content

Remove unused refetch vulnerabilities code

Daniel Tian requested to merge 217530-remove-unused-code into master

What does this MR do?

On the security dashboard, you can bulk dismiss vulnerabilities:

ksnip_20200921-230727

Currently, the code tries to refetch vulnerabilities after the bulk dismiss is finished in order to update the vulnerabilities. However, Apollo is smart enough to update its in-memory cache from the dismissal mutation, and also smart enough to not perform a network request despite the call to refetch():

ksnip_20200921-225240

This MR simply removes the code that attempts this, since it doesn't do anything anyway, and removing it preserves the same behavior.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Merge request reports

Loading