Differential D8425

Update `differential.close` to use DifferentialTransactionEditor
ClosedPublic
Actions

Authored by epriestley on Mar 6 2014, 4:33 PM.

Tags

None

Subscribers

Details

Reviewers

Maniphest Tasks

T2222: Implement ApplicationTransactions in Differential

Commits

Restricted Diffusion Commit
rP75514cc0f713: Update `differential.close` to use DifferentialTransactionEditor

Summary

Ref T2222. Straightforward update to new stuff.

Test Plan

Tried to close an uncloseable revision.
Closed a closeable revision.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan accepted this revision.Mar 6 2014, 10:43 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 6 2014, 10:43 PM

Closed by commit rP75514cc0f713 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

conduit/

ConduitAPI_differential_Method.php

3 lines

ConduitAPI_differential_close_Method.php

37 lines

transactions/

editor/

PhabricatorApplicationTransactionEditor.php

10 lines

Diff 20023

src/applications/differential/conduit/ConduitAPI_differential_Method.php

Loading...

src/applications/differential/conduit/ConduitAPI_differential_close_Method.php

Loading...

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...