Page MenuHomePhabricator

Modernize chatlog a bit
ClosedPublic

Authored by epriestley on Apr 10 2014, 4:20 PM.
Tags
None
Referenced Files
F18999544: D8746.id20748.diff
Nov 20 2025, 9:21 PM
F18877929: D8746.id20741.diff
Nov 6 2025, 12:58 PM
F18854825: D8746.id.diff
Nov 1 2025, 12:39 AM
F18852032: D8746.id20741.diff
Oct 31 2025, 6:20 AM
F18846543: D8746.diff
Oct 29 2025, 7:34 PM
F18844195: D8746.id20741.diff
Oct 29 2025, 3:54 AM
F18734755: D8746.diff
Sep 30 2025, 11:01 PM
F18664467: D8746.diff
Sep 24 2025, 12:00 AM
Subscribers

Details

Summary

Ref T4786. This doesn't fully fix the issue since there's no way to make channels public yet, but gets some of the infrastructure more up to date.

  • Allow public access to the list and log controllers.
  • Implement proper policy checks in the Events (this has no practical impact on the only controller that loads this stuff, it's just for general/future purposes).
  • Remove a old-style unused method for building page frames.
Test Plan

Viewed log list and log details as logged-in and logged out users.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Modernize chatlog a bit.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 10 2014, 6:30 PM
epriestley updated this revision to Diff 20748.

Closed by commit rPfacd3ea8ef5a (authored by @epriestley).