Skip to content

Generalize burnup chart service for other metrics

Heinrich Lee Yu requested to merge move-timebox-reports-graphql into master

What does this MR do?

This generalizes the burnup chart service because we would use the same data computed from iterating through events for other metrics.

This also moves the GraphQL burnup time series under a report field. The frontend will be changed to use this new endpoint in another MR so that we won't have problems during deploy. After those are deployed, we can then remove the old endpoint.

We are doing this now because this endpoint is still behind a feature flag and it is easier to change / remove this now before it becomes publicly available.

Related to #233707 (comment 413526459)

Screenshots

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
Edited by Heinrich Lee Yu

Merge request reports

Loading