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
Unknown Object (File)
Fri, Dec 20, 9:14 AM
Unknown Object (File)
Mon, Dec 9, 12:02 PM
Unknown Object (File)
Sat, Dec 7, 2:28 AM
Unknown Object (File)
Fri, Dec 6, 9:35 AM
Unknown Object (File)
Fri, Dec 6, 4:28 AM
Unknown Object (File)
Wed, Nov 27, 2:03 AM
Unknown Object (File)
Wed, Nov 27, 2:03 AM
Unknown Object (File)
Wed, Nov 27, 12:28 AM
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