Page MenuHomePhabricator

Fix Diffusion blame/highlight for logged-out users
ClosedPublic

Authored by epriestley on May 27 2014, 8:26 PM.
Tags
None
Referenced Files
F15481929: D9310.id22087.diff
Wed, Apr 9, 12:49 AM
F15455890: D9310.diff
Sun, Mar 30, 5:33 AM
F15449658: D9310.id22100.diff
Fri, Mar 28, 11:30 AM
F15449657: D9310.id22087.diff
Fri, Mar 28, 11:29 AM
F15405259: D9310.diff
Tue, Mar 18, 10:55 AM
F15400973: D9310.diff
Mon, Mar 17, 3:02 PM
F15385680: D9310.id22087.diff
Mar 14 2025, 11:09 PM
F15341016: D9310.id22087.diff
Mar 9 2025, 5:56 PM
Subscribers

Details

Summary

Fixes T5199. We try to save these options in user preferences, but logged-out users don't have preferences.

Instead, just use GET links for logged-out users.

Test Plan
  • As a logged-out user, toggled blame and highlight on and off.
  • As a logged-in user, toggled blame and highlight on and off.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Fix Diffusion blame/highlight for logged-out users.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: vrana, btrahan.
vrana edited edge metadata.
This revision is now accepted and ready to land.May 28 2014, 12:37 AM
epriestley updated this revision to Diff 22100.

Closed by commit rP2aef04a78afb (authored by @epriestley).