Page MenuHomePhabricator

Straighten out some stacked action removal behaviors
ClosedPublic

Authored by epriestley on Jan 5 2017, 3:15 AM.
Tags
None
Referenced Files
F19468977: D17143.id41230.diff
Sun, Jan 4, 1:35 PM
F19466993: D17143.diff
Sat, Jan 3, 12:23 PM
F19060082: D17143.id.diff
Nov 29 2025, 8:12 AM
F19054219: D17143.diff
Nov 28 2025, 11:23 AM
F18837979: D17143.id.diff
Oct 27 2025, 8:36 AM
F18834907: D17143.diff
Oct 26 2025, 11:51 AM
F18833673: D17143.diff
Oct 26 2025, 5:06 AM
F18692077: D17143.diff
Sep 26 2025, 10:11 PM
Subscribers
None

Details

Summary

Fixes T12065. Some of the normal behavior of these actions got juggled when I made "Accept" and "Reject" conflict.

Test Plan
  • Added "Accept".
  • Added "Reject", saw it remove "Accept".
  • Added "Change Projects".
  • Added "Assign/Claim".
  • Removed "Assign/Claim".
  • Removed "Accept".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jan 5 2017, 3:16 AM
This revision was automatically updated to reflect the committed changes.