Page MenuHomePhabricator
Differential D8443

Remove DifferentialCommentEditor and DifferentialCommentMail
ClosedPublic
Actions

Authored by epriestley on Mar 7 2014, 8:18 PM.
Tags
None
Referenced Files
F19064434: D8443.id.diff
Sat, Nov 29, 9:29 PM
F19057561: D8443.diff
Fri, Nov 28, 11:05 PM
F18846956: D8443.id.diff
Oct 29 2025, 10:10 PM
F18843643: D8443.diff
Oct 28 2025, 11:53 PM
F18826936: D8443.diff
Oct 24 2025, 6:39 AM
F18789091: D8443.id.diff
Oct 15 2025, 7:55 AM
F18682189: D8443.diff
Sep 26 2025, 5:21 AM
F18669747: D8443.id.diff
Sep 24 2025, 9:46 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T2222: Implement ApplicationTransactions in Differential
Commits
Restricted Diffusion Commit
rPf25cce1e695b: Remove DifferentialCommentEditor and DifferentialCommentMail
Summary

Ref T2222. These no longer have any callsites. Also got rid of a little bit of other code which also no longer has callsites.

Test Plan

grep

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

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 rPf25cce1e695b (authored by @epriestley).

Revision Contents
Changeset List

Diff 20045

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/editor/DifferentialCommentEditor.php

Loading...
View Options

src/applications/differential/editor/DifferentialRevisionEditor.php

Loading...
View Options

src/applications/differential/exception/DifferentialActionHasNoEffectException.php

Loading...
View Options

src/applications/differential/mail/DifferentialCommentMail.php

Loading...
Phabricator · Built by Phacility