Migrate security-report-download-dropdown in security_reports_app.vue to use newer merge-request-artifact-download component Part 1
As a result of:
Issue | Description |
---|---|
#289952 (closed) | Creates MR Artifact Download Component |
#328481 (closed) | Use MR Artifact Download component in Cov Fuzz and API Fuzz MR widgets |
#328194 (closed) | Creates Pipeline Artifact Download Component |
#328481 (closed) | Use Pipeline Artifact Component on Pipeline Security Tab |
We should go back and update for CE - Community Edition for SAST.
The security-report-download-dropdown
component has additional logic and a graphQL query that has already been wrapped in merge_request_artifact_download.vue
. We should swap out these components in favor of the newer one.
Part | Issue | Description |
---|---|---|
1 | #330429 (closed) | Move EE code to CE update existing usages <-- this one |
2 | #331025 (closed) | Swap out component for newer component |
Part 1 Implementation Plan
- Move the contents of https://gitlab.com/gitlab-org/gitlab/-/tree/master/ee/app/assets/javascripts/vue_shared/security_reports/components/artifact_downloads into the CE directory
- Refactor imports and usages of merge_request_aritact_downloads.js and pipeline_artifact_downloads.vue to point to CE directory
- Achieve passing pipeline
Part 2
Edited by -