Differential D17118

Restore some minor state behaviors to Differential on EditEngine
ClosedPublic
Actions

Authored by epriestley on Dec 31 2016, 5:24 PM.

Tags

None

Referenced Files

	F15440795: D17118.id.diff
	Wed, Mar 26, 2:36 PM

	F15434023: D17118.id41175.diff
	Tue, Mar 25, 1:33 AM

	F15433984: D17118.diff
	Tue, Mar 25, 1:25 AM

	F15430559: D17118.id.diff
	Mon, Mar 24, 8:11 AM

	F15407141: D17118.id41162.diff
	Tue, Mar 18, 4:03 PM

	F15377532: D17118.id41162.diff
	Thu, Mar 13, 9:27 AM

	F15354568: D17118.diff
	Tue, Mar 11, 4:45 AM

	Unknown Object (File)
	Feb 18 2025, 12:16 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T11114: Move Differential to EditEngine

Commits

rPb373dcef7448: Restore some minor state behaviors to Differential on EditEngine

Summary

Ref T11114. This restores:

Commandeering should exeucte Herald.
Commandeering should swap reviewers.
"Request Review" on an "Accepted" revision should downgrade reviewers so they have to accept again.

Test Plan

Commandeered, saw Herald run and reviewers swap.
Requested review of an accepted revision, saw it drop down to "Needs Review" with "Accepted Prior" on the reviewer.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Dec 31 2016, 5:24 PM

chad accepted this revision.Dec 31 2016, 6:03 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Dec 31 2016, 6:03 PM

Closed by commit rPb373dcef7448: Restore some minor state behaviors to Differential on EditEngine (authored by epriestley, committed by epriestley). · Explain WhyDec 31 2016, 6:11 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

editor/

DifferentialTransactionEditor.php

98 lines

transactions/

editengine/

PhabricatorEditEngine.php

4 lines

Diff 41175

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...

src/applications/transactions/editengine/PhabricatorEditEngine.php

Loading...