Page MenuHomePhabricator
Differential D17107

Restore "Reclaim" and "Abandon" actions to Differential on EditEngine
ClosedPublic
Actions

Authored by epriestley on Dec 28 2016, 8:53 PM.
Tags
None
Referenced Files
F15508023: D17107.diff
Wed, Apr 16, 1:03 AM
F15486350: D17107.id41145.diff
Thu, Apr 10, 6:35 AM
F15475930: D17107.id41166.diff
Mon, Apr 7, 1:14 AM
F15446519: D17107.id41146.diff
Thu, Mar 27, 7:25 PM
F15446214: D17107.id.diff
Thu, Mar 27, 5:45 PM
F15442332: D17107.diff
Wed, Mar 26, 10:49 PM
F15421812: D17107.id41146.diff
Mar 22 2025, 2:21 AM
F15409322: D17107.id41166.diff
Mar 19 2025, 4:10 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T11114: Move Differential to EditEngine
Commits
rP3c5a17ba8a55: Restore "Reclaim" and "Abandon" actions to Differential on EditEngine
Summary

Ref T11114. This begins restoring comment actions to Differential, but on top of EditEngine.

Test Plan

Screen Shot 2016-12-28 at 12.49.26 PM.png (418×1 px, 44 KB)

Diff Detail

Repository
rP Phabricator
Branch
haunt3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15032
Build 19721: Run Core Tests
Build 19720: arc lint + arc unit

Event Timeline

epriestley created this revision.Dec 28 2016, 8:53 PM
Herald edited edge metadata. · View Herald TranscriptDec 28 2016, 8:53 PM
epriestley updated this revision to Diff 41146.Dec 28 2016, 9:10 PM
  • Prevent these actions from generating transactions during web form submission.
chad accepted this revision.Dec 29 2016, 5:47 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 29 2016, 5:47 PM
Closed by commit rP3c5a17ba8a55: Restore "Reclaim" and "Abandon" actions to Differential on EditEngine (authored by epriestley, committed by epriestley). · Explain WhyDec 31 2016, 6:06 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
18 lines
src/
10 lines
applications/
differential/
editor/
8 lines
storage/
5 lines
xaction/
67 lines
88 lines
62 lines
transactions/
commentaction/
28 lines
editfield/
34 lines
webroot/
rsrc/
css/
phui/
5 lines
js/
phuix/
22 lines

Diff 41145

View Options

resources/celerity/map.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/editor/DifferentialRevisionEditEngine.php

Loading...
View Options

src/applications/differential/storage/DifferentialRevision.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionAbandonTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionActionTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionReclaimTransaction.php

Loading...
View Options

src/applications/transactions/commentaction/PhabricatorEditEngineStaticCommentAction.php

Loading...
View Options

src/applications/transactions/editfield/PhabricatorApplyEditField.php

Loading...
View Options

webroot/rsrc/css/phui/phui-form-view.css

Loading...
View Options

webroot/rsrc/js/phuix/PHUIXFormControl.js

Loading...
Phabricator · Built by Phacility