Page MenuHomePhabricator

Order actions sensibly within Differential revision comment action groups
ClosedPublic

Authored by epriestley on Dec 29 2016, 9:44 PM.
Tags
None
Referenced Files
F18805328: D17115.id41172.diff
Sat, Oct 18, 3:05 PM
F18761142: D17115.diff
Mon, Oct 6, 12:30 PM
F18759868: D17115.id.diff
Mon, Oct 6, 6:25 AM
F18756616: D17115.diff
Sun, Oct 5, 12:23 PM
F18681890: D17115.id41158.diff
Fri, Sep 26, 5:01 AM
F18680781: D17115.id41158.diff
Fri, Sep 26, 3:07 AM
F18671807: D17115.id41172.diff
Thu, Sep 25, 5:23 AM
F18653149: D17115.diff
Sep 21 2025, 11:22 AM
Subscribers
None

Details

Summary

Ref T11114. See D17114 for some discussion.

For review actions: accept, reject, resign.

For revision actions, order is basically least-severe to most-severe action pairs: plan changes, request review, close, reopen, abandon, reclaim, commandeer.

Test Plan

Viewed revisions as an author and a reviewer, saw sensible action order within action groups.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad edited edge metadata.
This revision is now accepted and ready to land.Dec 30 2016, 5:56 PM
This revision was automatically updated to reflect the committed changes.