Resolve "CI view for SAST"
What does this MR do?
Frontend:
- Moves security report code to shared folder.
- Creates vue_shared folder under ee for security_reports
- Creates a mixin with the common app methods (mr widget & pipelines tab)
- Creates a utils file with all the common store methods
- Moves the CSS into a different file
- Renames all CSS classes from
mr-widget
toreport-block
- Adds support in the pipelines page to mount security reports app
- Creates app to support more than one report, but only adds SAST now
- Fixes typo in the name of a file
- Adds check for not always render the collapse button
- Add tests for the new tab
- Create CE port - https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/17176
- Adds Vue app to render the summary text in the pipeline widget
- Updates the counter in the security tab
Screenshots (if relevant)
no vulnerabilities | with vulnerabilities |
---|---|
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary -
Documentation created/updated -
API support added -
Tests added for this feature/bug - Review
-
Has been reviewed by UX -
Has been reviewed by Frontend -
Has been reviewed by Backend -
Has been reviewed by Database
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Squashed related commits together -
Internationalization required/considered -
If paid feature, have we considered GitLab.com plan and how it works for groups and is there a design for promoting it to users who aren't on the correct plan -
End-to-end tests pass ( package-qa
manual pipeline job)
What are the relevant issue numbers?
Closes #3776 (closed)
Edited by Chris Peressini