Page MenuHomePhabricator

Remove the ability to disable blame in Diffusion
ClosedPublic

Authored by epriestley on Apr 5 2018, 2:33 PM.

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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Apr 5 2018, 2:33 PM
epriestley requested review of this revision.Apr 5 2018, 2:34 PM
mydeveloperday accepted this revision.Apr 5 2018, 5:06 PM
This revision is now accepted and ready to land.Apr 5 2018, 5:06 PM
This revision was automatically updated to reflect the committed changes.