Page MenuHomePhabricator

Separate revision updates into two separate comments
ClosedPublic

Authored by epriestley on Feb 11 2014, 10:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 7:01 AM
Unknown Object (File)
Tue, Apr 9, 11:57 PM
Unknown Object (File)
Fri, Apr 5, 6:28 AM
Unknown Object (File)
Sun, Mar 31, 6:24 PM
Unknown Object (File)
Fri, Mar 22, 2:39 AM
Unknown Object (File)
Fri, Mar 22, 2:39 AM
Unknown Object (File)
Fri, Mar 22, 1:55 AM
Unknown Object (File)
Fri, Mar 22, 1:16 AM
Subscribers

Details

Summary

Ref T2222. Instead of writing one comment which performs both a diff update and adds a comment, write two comments, one for each action. These will translate directly into ApplicationTransactions writes.

This has a small impact on the UX: these updates now render in two rows, instead of one. After T2222, they'll automerge back together.

{F111909}

Test Plan

Updated a revision.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped