Page MenuHomePhabricator
Differential D8236

Render Differential previews through ApplicationTransactions
ClosedPublic
Actions

Authored by epriestley on Feb 14 2014, 9:08 PM.
Tags
None
Referenced Files
F18106972: D8236.id.diff
Sun, Aug 10, 11:34 PM
F17926207: D8236.id19600.diff
Wed, Jul 30, 12:46 PM
F17902087: D8236.id.diff
Tue, Jul 29, 10:08 AM
F17869846: D8236.diff
Mon, Jul 28, 12:24 PM
F17849828: D8236.diff
Sun, Jul 27, 9:09 AM
F17747606: D8236.diff
Mon, Jul 21, 10:27 PM
Unknown Object (File)
Jun 29 2025, 12:32 AM
Unknown Object (File)
Jun 28 2025, 12:34 PM
View All Files
Subscribers
aran
Tokens
"Piece of Eight" token, awarded by btrahan.

Details

Reviewers
btrahan
Maniphest Tasks
T2222: Implement ApplicationTransactions in Differential
Commits
Restricted Diffusion Commit
rP23146f2bb9e7: Render Differential previews through ApplicationTransactions
Summary

Ref T2222. Use new code for rendering. Delete DifferentialRevisionCommentView, which has no remaining callsites.

Test Plan

Went through all the different actions and verified the previews rendered correctly. Reloaded page to test draft behavior.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
differential/
controller/
104 lines
view/

Diff 19600

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/controller/DifferentialCommentPreviewController.php

Loading...
View Options

src/applications/differential/view/DifferentialRevisionCommentView.php

Loading...
Phabricator · Built by Phacility