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
F18431232: D9310.id.diff
Sun, Aug 31, 4:59 AM
F18419728: D9310.diff
Sat, Aug 30, 2:54 PM
F18301486: D9310.id22100.diff
Sat, Aug 23, 8:02 PM
F18295403: D9310.id22087.diff
Sat, Aug 23, 2:50 PM
F18053625: D9310.id22100.diff
Aug 4 2025, 5:16 AM
F18053450: D9310.id22087.diff
Aug 4 2025, 5:03 AM
F17922227: D9310.id.diff
Jul 30 2025, 6:28 AM
F17912565: D9310.diff
Jul 29 2025, 4:27 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).