Page MenuHomePhabricator

Straighten out some stacked action removal behaviors
ClosedPublic

Authored by epriestley on Jan 5 2017, 3:15 AM.
Tags
None
Referenced Files
F19060082: D17143.id.diff
Sat, Nov 29, 8:12 AM
F19054219: D17143.diff
Fri, Nov 28, 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
F18105649: D17143.id41229.diff
Aug 10 2025, 7:00 PM
F18101290: D17143.id41230.diff
Aug 9 2025, 12:25 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.