Page MenuHomePhabricator

Consolidate burndown logic into a "BurndownChartEngine"
ClosedPublic

Authored by epriestley on May 3 2019, 7:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 30, 5:00 PM
Unknown Object (File)
Wed, Jan 29, 8:21 AM
Unknown Object (File)
Tue, Jan 28, 5:15 AM
Unknown Object (File)
Tue, Jan 28, 5:15 AM
Unknown Object (File)
Tue, Jan 28, 5:15 AM
Unknown Object (File)
Sat, Jan 25, 4:33 PM
Unknown Object (File)
Sat, Jan 25, 5:01 AM
Unknown Object (File)
Sat, Jan 25, 5:01 AM
Subscribers

Details

Summary

Ref T13279. For now, we need to render burndowns from both Maniphest (legacy) and Projects (new prototype).

Consolidate this logic into a "BurndownChartEngine". I plan to expand this to work a bit like a "SearchEngine", and serve as a UI layer on top of the raw chart features.

The old "ChartEngine" is now "ChartRenderingEngine".

Test Plan
  • Viewed burndowns ("burnups") in Maniphest.
  • Viewed burndowns in Projects.
  • Saw the same chart.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable