Follow-up from "Enable group level vulnerability report grouping"
The following discussion from !137778 (merged) should be addressed:
-
@lorenzvanherwaarden started a discussion:
suggestion: now that we're adding this provide/inject for the group vulnerability report, it crosses my mind that we also have a
provide
coming from the highest levelvulnerability_report_init.js
which isdashboardType
. This can be'project'
,'group'
,'pipeline'
, or'instance'
. WDYT about using that here instead? This would also meanisProjectVulnerabilityReport
can be removed.We can also do this in a follow-up if you'd like.
note: reminder to update the level in https://gitlab.com/gitlab-org/gitlab/-/blob/6dab8c151f52e23abd0eed6b6d7fd7370b59f429/ee/app/assets/javascripts/security_dashboard/components/shared/vulnerability_report/vulnerability_report.vue#L348 accordingly