Follow-up from "Move project level vulnerabilities to vulnerability report component"
The following discussion from !59586 (merged) should be addressed:
-
@dftian started a discussion: (+1 comment) improvement (optional): I think this is fine as-is, but because the two variables are provided, only the count list needs the computed property, and all we're really doing is passing it down rather than using it in this component:
<vulnerabilities-count-list :full-path="fullPath" :filters="filters" />
We can probably inject
groupFullPath
andprojectFullPath
intoVulnerabilitiesCountList
and move the computed property there (or just dofullPath: this.groupFullPath || this.projectFullPath
in its query'svariables()
).