Show security reports summary in pipelines' Security Dashboard - Frontend
frontend Implementation parent issue for #197115 (closed)
Problem to solve
This is a 2nd iteration issue for #197115 (closed) to move the data into an expandable component.
Implementation Strategy
-
Cleanup the work that was done in !31136 (diffs) -
Revert the changes done in ee/app/assets/javascripts/security_dashboard/components/filter.vue -
Revert the changes done in ee/app/assets/javascripts/security_dashboard/components/filters.vueand its specs -
Stop passing the reports summary down the filters in ee/app/assets/javascripts/security_dashboard/components/security_dashboard_vuex.vue
-
-
Update the GraphQL query in the frontend to match the final schema that was implemented in !31550 (merged). -
Create a new SecurityReportsSummarycomponent-
The component is based on GlCollapse -
We'll want to store the collapsed/expanded state in the local storage -
The component receives securityReportSummaryas a prop- note: we could potentially move the GraphQL query to the new component, but keeping it in the parent component makes it easier to reuse if we ever wanted to display similar summaries somewhere else.
-
Hide summary entries if the corresponding scanner is nullin the GraphQL response (= scanner is not setup). The backend for this will be done in #223073 (closed) -
Add unit tests
-
-
Extend the PipelineSecurityDashboardcomponent's template to include theSecurityReportsSummarycomponent
Edited by Paul Gascou-Vaillancourt
