Differential D20879

Make the top-level burndown chart in "Maniphest > Reports" show open tasks, not total tasks
ClosedPublic
Actions

Authored by epriestley on Oct 29 2019, 8:46 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Dec 8, 8:26 AM

	Unknown Object (File)
	Thu, Dec 5, 11:53 AM

	Unknown Object (File)
	Wed, Dec 4, 1:42 PM

	Unknown Object (File)
	Sun, Nov 24, 10:03 PM

	Unknown Object (File)
	Nov 21 2024, 5:04 PM

	Unknown Object (File)
	Nov 21 2024, 2:02 PM

	Unknown Object (File)
	Nov 16 2024, 10:12 PM

	Unknown Object (File)
	Nov 16 2024, 5:33 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13279: Build Charting for Facts
Commits: rP9d8cdce8e1f2: Make the top-level burndown chart in "Maniphest > Reports" show open tasks, not…

Summary

Ref T13279. See PHI1491. Currently, the top-level "Burnup Rate" chart in Maniphest shows total created tasks above the X-axis, without adjusting for closures.

This is unintended and not very useful. The filtered-by-project charts show the right value (cumulative open tasks, i.e. open minus close). Change the value to aggregate creation events and status change events.

Test Plan

Viewed top-level chart, saw the value no longer monotonically increasing.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Oct 29 2019, 8:46 PM

Harbormaster completed remote builds in B23604: Diff 49775.Oct 29 2019, 8:48 PM

This revision was not accepted when it landed; it landed in state Needs Review.Oct 29 2019, 8:48 PM

epriestley requested review of this revision.

Closed by commit rP9d8cdce8e1f2: Make the top-level burndown chart in "Maniphest > Reports" show open tasks, not… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP9d8cdce8e1f2: Make the top-level burndown chart in "Maniphest > Reports" show open tasks, not….

Revision Contents
Changeset List

Path

Size

src/

applications/

project/

chart/

PhabricatorProjectBurndownChartEngine.php

6 lines

Diff 49776

src/applications/project/chart/PhabricatorProjectBurndownChartEngine.php

Loading...