Page MenuHomePhabricator
Differential D8442

Use TransactionEditor in differential.createcomment
ClosedPublic
Actions

Authored by epriestley on Mar 7 2014, 8:04 PM.
Tags
None
Referenced Files
F13163374: D8442.diff
Mon, May 6, 7:37 PM
Unknown Object (File)
Sun, Apr 28, 6:00 AM
Unknown Object (File)
Sat, Apr 27, 6:50 PM
Unknown Object (File)
Sat, Apr 27, 6:49 PM
Unknown Object (File)
Sat, Apr 20, 6:35 PM
Unknown Object (File)
Thu, Apr 11, 11:08 AM
Unknown Object (File)
Mon, Apr 8, 7:19 PM
Unknown Object (File)
Mon, Apr 8, 7:19 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T2222: Implement ApplicationTransactions in Differential
Commits
Restricted Diffusion Commit
rP1c51ed594065: Use TransactionEditor in differential.createcomment
Summary

Ref T2222. Update this callsite; pretty straightforward.

Test Plan

Used Conduit to take actions and saw their effects in Differential.

Diff Detail

Repository
rP Phabricator
Branch
dx6
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/differential/query/DifferentialTransactionQuery.php:14XHP16TODO Comment
Unit
Tests Passed

Event Timeline

btrahan accepted this revision.Mar 8 2014, 12:26 AM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 8 2014, 12:26 AM
epriestley closed this revision.Mar 8 2014, 1:44 AM

Closed by commit rP1c51ed594065 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
conduit/
55 lines
controller/
40 lines
query/
35 lines
transactions/
editor/
22 lines

Diff 20037

View Options

src/applications/differential/conduit/ConduitAPI_differential_createcomment_Method.php

Loading...
View Options

src/applications/differential/controller/DifferentialCommentSaveController.php

Loading...
View Options

src/applications/differential/query/DifferentialTransactionQuery.php

Loading...
View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...
Phabricator · Built by Phacility