Page MenuHomePhabricator

Move Differential to EditEngine comments
ClosedPublic

Authored by epriestley on Dec 28 2016, 6:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 10:46 AM
Unknown Object (File)
Sun, Dec 15, 1:58 AM
Unknown Object (File)
Tue, Dec 10, 5:10 PM
Unknown Object (File)
Sun, Dec 8, 8:27 PM
Unknown Object (File)
Tue, Dec 3, 8:33 AM
Unknown Object (File)
Sun, Dec 1, 6:05 PM
Unknown Object (File)
Fri, Nov 29, 8:32 PM
Unknown Object (File)
Wed, Nov 27, 4:50 AM
Subscribers
None

Details

Summary

Ref T11114. This is a transitional change that breaks a bunch of stuff. I'll hold it until I've restored features.

This stuff works:

  • Commenting.
  • Subscribers/tags/reviewers.
  • Pinning.
  • Drafts.

This stuff does not work yet:

  • Preview of inline comments.
  • Probably submitting inlines, whatsoever.
  • Comment-area warnings like "There are failing tests."
  • All meaningful actions (accept, reject, etc).
Test Plan

Commented on a revision. Essentially nothing else works yet.

Diff Detail

Repository
rP Phabricator
Branch
haunt2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15031
Build 19719: Run Core Tests
Build 19718: arc lint + arc unit