Page MenuHomePhabricator

Give Phriction documents a normal timeline
ClosedPublic

Authored by epriestley on Sep 11 2018, 3:15 PM.
Tags
None
Referenced Files
F14493022: D19659.diff
Fri, Jan 3, 12:00 AM
Unknown Object (File)
Sat, Dec 21, 9:41 AM
Unknown Object (File)
Fri, Dec 20, 9:48 PM
Unknown Object (File)
Mon, Dec 16, 6:21 PM
Unknown Object (File)
Sat, Dec 14, 1:35 PM
Unknown Object (File)
Mon, Dec 9, 11:41 AM
Unknown Object (File)
Sat, Dec 7, 1:45 AM
Unknown Object (File)
Thu, Dec 5, 8:39 PM
Subscribers
None

Details

Summary

Ref T13077. See PHI840. Ref T1894. I'm planning to just let you comment on Phriction documents. I think this will create a few problems (e.g., around popular documents which collect long comment threads that are eventually obsolete) but nothing should be too terribly critical (e.g., we handle it gracefully when objects have very large number of comments/transactions) and for most documents this is likely just a net improvement.

"Just enable comments" is probably not the final iteration on this, but I think it's probably a step forward on the balance, not a step sideways or a slippery slope down into a dark hole or anything.

Test Plan

Screen Shot 2018-09-11 at 8.10.33 AM.png (1×1 px, 161 KB)

Diff Detail

Repository
rP Phabricator
Branch
wiki1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20789
Build 28272: Run Core Tests
Build 28271: arc lint + arc unit