Page MenuHomePhabricator

Implement "Group by Owner" on Workboards
ClosedPublic

Authored by epriestley on Mar 11 2019, 5:01 AM.
Tags
None
Referenced Files
F14089400: D20270.diff
Sun, Nov 24, 10:12 AM
F14088195: D20270.id48385.diff
Sun, Nov 24, 1:58 AM
Unknown Object (File)
Fri, Nov 22, 2:21 AM
Unknown Object (File)
Wed, Nov 20, 8:36 AM
Unknown Object (File)
Fri, Nov 15, 11:45 PM
Unknown Object (File)
Tue, Nov 12, 1:30 AM
Unknown Object (File)
Mon, Nov 11, 4:56 AM
Unknown Object (File)
Fri, Nov 8, 4:02 PM
Subscribers
None
Tokens
"Love" token, awarded by leoluk.

Details

Summary

Depends on D20269. Ref T10333. Now that orderings are modularized, this is fairly easy to implement. This isn't super fancy for now (e.g., no profile images) but I'll touch it up in a general polish followup.

Test Plan

Screen Shot 2019-03-10 at 10.00.50 PM.png (629×380 px, 40 KB)

Diff Detail

Repository
rP Phabricator
Branch
grip6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22248
Build 30421: Run Core Tests
Build 30420: arc lint + arc unit

Event Timeline

amckinley added inline comments.
src/applications/project/order/PhabricatorProjectColumnOwnerOrder.php
72

"by the owner's"

81–83

Maybe sort them all into a single "Invalid Owner" bucket?

This revision is now accepted and ready to land.Mar 12 2019, 7:53 PM
  • Typo correction.

I think the "Invalid Owner" case is likely very rare and having more information may be helpful in sorting it out, but there's no reason we couldn't do a single "invalid" bucket if that doesn't prove to be the case.

Other groupings like "priority" and "status" can end up with database values that have no display value, and I think the extra headers are probably helpful on the balance there, since sometimes they point directly at a corrective action (usually: reverting changes to configuration to put back priorities or statuses that you incorrectly got rid of). In these cases, at least, it seems more helpful to have an Unknown Status ("waylayed") header than an Invalid Status header, although admittedly it's less likely that anyone can take specific corrective action based on Unknown User ("PHID-USER-xyz") vs Invalid Owner.

This revision was automatically updated to reflect the committed changes.