Page MenuHomePhabricator

Remove the ability to disable blame in Diffusion
ClosedPublic

Authored by epriestley on Apr 5 2018, 2:33 PM.
Tags
None
Referenced Files
F15448179: D19300.diff
Fri, Mar 28, 3:21 AM
F15445792: D19300.diff
Thu, Mar 27, 4:02 PM
F15436476: D19300.diff
Tue, Mar 25, 2:30 PM
F15435236: D19300.diff
Tue, Mar 25, 7:28 AM
F15431314: D19300.id46201.diff
Mon, Mar 24, 11:44 AM
F15427082: D19300.id.diff
Sun, Mar 23, 12:39 PM
F15342835: D19300.id46188.diff
Sun, Mar 9, 9:30 PM
Unknown Object (File)
Feb 20 2025, 1:39 AM
Subscribers
None

Details

Summary

Ref T13105. Given that we now load blame with AJAX, it's not clear that there's any benefit to disabling it. This would also interact oddly with the document engine.

Test Plan

Viewed files in Diffusion, no longer saw blame-related options.

Diff Detail

Repository
rP Phabricator
Branch
dfile1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20010
Build 27132: Run Core Tests
Build 27131: arc lint + arc unit