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)
Wed, Apr 17, 2:59 PM
Unknown Object (File)
Tue, Apr 16, 10:19 AM
Unknown Object (File)
Fri, Apr 12, 9:32 AM
Unknown Object (File)
Thu, Apr 11, 7:28 AM
Unknown Object (File)
Fri, Apr 5, 3:40 PM
Unknown Object (File)
Fri, Mar 29, 4:57 PM
Unknown Object (File)
Feb 11 2024, 6:32 PM
Unknown Object (File)
Feb 10 2024, 3:27 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