Skip to content

POC: Add chart to report page

Brandon Labuschagne requested to merge add-chart-api-call-to-report-oages into master

What does this MR do?

This MR introduces a chart to the reports page. It is still POC feature with a number of items which require further development. This is simply an iteration forward. The feature vision can be seen in this epic &3214 (closed)

Screenshots

Screenshot_2020-07-14_at_12.43.48

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team

Testing

In order to test you will need to enable the group_activity_analytics and report_pages feature flags. You can then browse to a group overview page and click the highlighted link as displayed in the description. The data being retrieved is currently mocked on the backend so the results displayed in the chart will be the same regardless of the group used for navigation.

Related issue

#218460 (closed)

Edited by Brandon Labuschagne

Merge request reports