Skip to content

Hide burndown toggle switch for new milestones

Simon Knox requested to merge psi-burndown-tidier-toggle into master

What does this MR do?

In !42314 (merged) we added an ugly switch to switch between event-based graphql data for burndown and the mutable REST endpoint. We don't need this for new charts, as they always have resource event data.

Add a check for the first ResourceStateEvent, and if the milestone is created after that, don't show the toggle.

Also hide the yellow warning box about old events/issues, as that is only applicable for issues created before 9.5

Screenshots

before after
Screen_Shot_2020-09-24_at_5.24.28_pm Screen_Shot_2020-09-24_at_5.21.12_pm

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 Simon Knox

Merge request reports