Skip to content

Resolve "Add generic 'metrics' report type to merge requests" frontend

What does this MR do?

Implement the frontend custom metrics on the merge request page.

No Changes

Screen_Shot_2019-03-28_at_11.22.35_AM

One Change

Screen_Shot_2019-03-28_at_11.23.19_AM

Two Changes

Screen_Shot_2019-03-28_at_11.23.59_AM

Overflowing List

Screen_Shot_2019-03-28_at_11.20.39_AM

Loading List

Screen_Shot_2019-04-01_at_3.47.27_PM

Error loading list

Screen_Shot_2019-03-28_at_12.01.26_PM

New and Removed Metrics

Screen_Shot_2019-04-02_at_2.53.50_PM

  • Create ee/app/assets/javascripts/vue_shared/custom_metrics/ folder containing:
    • grouped_custom_metrics_app.vue
    • /store similar to ee/app/assets/javascripts/vue_shared/security_reports/store
  • reuse report components from app/assets/javascripts/reports/components
  • add grouped_custom_metrics_app component to ee/app/assets/javascripts/vue_merge_request_widget/mr_widget_options.vue
  • pass the endpoint to the grouped_custom_metrics_app component
  • Create spec for grouped_custom_metrics_app and its store

What are the relevant issue numbers?

#9788 (closed)

Does this MR meet the acceptance criteria?

Edited by James Heimbuck

Merge request reports