Differential D17143

Straighten out some stacked action removal behaviors
ClosedPublic
Actions

Authored by epriestley on Jan 5 2017, 3:15 AM.

Details

Reviewers

chad

Maniphest Tasks

T12065: Can't close actions in Maniphest

Commits

rP4b94ae15e0bc: Straighten out some stacked action removal behaviors

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

Branch

action1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15112
Build 19841: Run Core Tests
Build 19840: arc lint + arc unit

Event Timeline

epriestley created this revision.Jan 5 2017, 3:15 AM

chad accepted this revision.Jan 5 2017, 3:16 AM

This revision is now accepted and ready to land.Jan 5 2017, 3:16 AM

Harbormaster completed remote builds in B15112: Diff 41229.Jan 5 2017, 3:16 AM

Closed by commit rP4b94ae15e0bc: Straighten out some stacked action removal behaviors (authored by epriestley, committed by epriestley). · Explain WhyJan 5 2017, 3:17 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

22 lines

webroot/

rsrc/

js/

application/

transactions/

behavior-comment-actions.js

6 lines

Diff 41229

View Options

resources/celerity/map.php