Skip to content

Resolve "CI view for SAST"

Dmytro Zaporozhets (DZ) requested to merge 3776-ci-view-for-sast into master

What does this MR do?

Frontend:

  1. Moves security report code to shared folder.
  2. Creates vue_shared folder under ee for security_reports
  3. Creates a mixin with the common app methods (mr widget & pipelines tab)
  4. Creates a utils file with all the common store methods
  5. Moves the CSS into a different file
  6. Renames all CSS classes from mr-widget to report-block
  7. Adds support in the pipelines page to mount security reports app
  8. Creates app to support more than one report, but only adds SAST now
  9. Fixes typo in the name of a file
  10. Adds check for not always render the collapse button
  11. Add tests for the new tab
  12. Create CE port - https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/17176
  13. Adds Vue app to render the summary text in the pipeline widget
  14. Updates the counter in the security tab

Screenshots (if relevant)

no vulnerabilities with vulnerabilities
Screen_Shot_2018-02-15_at_21.39.13 Screen_Shot_2018-02-16_at_15.18.46

Screen_Shot_2018-02-19_at_17.42.55

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #3776 (closed)

Edited by Chris Peressini

Merge request reports

Loading