Page MenuHomePhabricator

Implement "Group by Owner" on Workboards

Authored by epriestley on Mar 11 2019, 5:01 AM.



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

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

amckinley added inline comments.

"by the owner's"


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.