Page MenuHomePhabricator
Differential D9136

Move activity log rendering to a dashboard panel
ClosedPublic
Actions

Authored by epriestley on May 15 2014, 4:58 PM.
Tags
None
Referenced Files
F18774162: D9136.diff
Thu, Oct 9, 3:35 PM
F18734863: D9136.id.diff
Tue, Sep 30, 11:04 PM
F18683657: D9136.id.diff
Fri, Sep 26, 7:58 AM
F18672172: D9136.id21723.diff
Thu, Sep 25, 6:53 AM
F18625295: D9136.id.diff
Sep 15 2025, 10:25 PM
F18616318: D9136.diff
Sep 14 2025, 5:06 PM
F18557667: D9136.id.diff
Sep 8 2025, 8:04 PM
F18514387: D9136.diff
Sep 5 2025, 10:11 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T4986: Implement new Dashboard panel types
Commits
Restricted Diffusion Commit
rPe6f6a58f93c1: Move activity log rendering to a dashboard panel
Summary

Ref T4986. Swap this in. Two minor notes:

  • I adjusted the SearchEngine to add an additional constraint when the viewer isn't an admin. This mostly stops us from doing a bunch of unnecessary work.
  • I fixed the settings panel to paginate (currently loads all results, slow in production).
Test Plan

Viewed logs; viewed settings panel; created a dashboard panel.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 21704.May 15 2014, 4:58 PM
epriestley retitled this revision from to Move activity log rendering to a dashboard panel.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T4986: Implement new Dashboard panel types.
Herald added a subscriber: epriestley. · View Herald TranscriptMay 15 2014, 4:58 PM
Harbormaster completed remote builds in B487: Diff 21704.May 15 2014, 4:59 PM
btrahan accepted this revision.May 15 2014, 7:43 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.May 15 2014, 7:43 PM
epriestley closed this revision.May 16 2014, 2:17 AM
epriestley updated this revision to Diff 21723.

Closed by commit rPe6f6a58f93c1 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
people/
controller/
32 lines
query/
48 lines
settings/
panel/
11 lines

Diff 21723

View Options

src/applications/people/controller/PhabricatorPeopleLogsController.php

Loading...
View Options

src/applications/people/query/PhabricatorPeopleLogSearchEngine.php

Loading...
View Options

src/applications/settings/panel/PhabricatorSettingsPanelActivity.php

Loading...
Phabricator · Built by Phacility