Skip to content

Move to supporting multiple charts per page for Insights

Mark Fletcher requested to merge insights-multi-charts-per-page into master

What does this MR do?

  1. Move to supporting multiple charts per page for Insights
  • Config file change
    • No longer accepts single chart, array only
    • Display warning for old config detected
  • Loads new charts upon tab change
    • Renders those successful
    • Flash message for errors
    • Summary of error instead of chart
  1. Announce warning that there was a problem reading config file
  2. Announce errors loading individual charts
  3. Move to a dropdown instead of a tabbed page picker
  • Better accommodates a large list of charts

Multi charts preview

insights-multi-charts

Invalid config preview

insights-errors-invalid-config-yml

Chart loading errors

insights-errors-unknown-chart

insights-errors-unknown-issuable-type


  • Docs change required
  • Replace Docs images for dropdowns required
  • Change to default config required

What are the relevant issue numbers?

Closes #10720 (closed) Closes #10466 (closed)

Does this MR meet the acceptance criteria?

Edited by Mark Fletcher

Merge request reports