Skip to content

Feat(analytics/insights): load charts individually

What does this MR do?

This MR makes it so that charts on the analytics/insights page loads individually, instead of waiting for all data to load before displaying the page. See screenshots

Screenshots

Before After (simulated different load times and error) After (no simulation)
Screen_Recording_2020-06-10_at_16.00.55 Screen_Recording_2020-06-10_at_15.59.10 Screen_Recording_2020-06-10_at_16.11.13

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

This change impacts the charts on the insights page and its charts. Worst-case if these changes cause a chart not to render, at least other charts would be able to render properly. Though, these changes are minimal, so I don't see a huge risk for that happening.

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

Related to #219742 (closed)

Edited by Michael Lunøe

Merge request reports