Page MenuHomePhabricator

Track closed date and closing user for tasks explicitly
ClosedPublic

Authored by epriestley on Feb 8 2018, 10:46 PM.
Tags
None
Referenced Files
F13092743: D19037.id45645.diff
Thu, Apr 25, 5:02 AM
F13092583: D19037.diff
Thu, Apr 25, 3:54 AM
F13087424: D19037.diff
Thu, Apr 25, 12:58 AM
Unknown Object (File)
Fri, Apr 19, 7:18 PM
Unknown Object (File)
Fri, Apr 19, 12:59 AM
Unknown Object (File)
Thu, Apr 18, 12:41 PM
Unknown Object (File)
Mon, Apr 15, 1:29 PM
Unknown Object (File)
Sun, Apr 14, 9:34 AM
Subscribers
None

Details

Summary

Ref T4434. Although some of the use cases for this data are better fits for Facts, this data is reasonable to track separately.

I have an approximate view of it already ("closed, ordered by date modified") that's useful to review things that were fixed recently. This lets us make that view more effective.

This just adds (and populates) the storage. Followups will add Conduit, Export, Search, and UI support.

This is slightly tricky because merges work oddly (see T13020).

Test Plan
  • Ran migration, checked database for sensible results.
  • Created a task in open/closed status, got the right database values.
  • Modified a task to close/open it, got the right values.
  • Merged an open task, got updates.

Diff Detail

Repository
rP Phabricator
Branch
task1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19441
Build 26297: Run Core Tests
Build 26296: arc lint + arc unit