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
F18140400: D9136.id21704.diff
Thu, Aug 14, 4:08 AM
F18111461: D9136.diff
Tue, Aug 12, 11:03 AM
F18110537: D9136.id.diff
Mon, Aug 11, 8:50 PM
F18108118: D9136.id21723.diff
Mon, Aug 11, 5:02 AM
F18105863: D9136.diff
Sun, Aug 10, 8:06 PM
F18089522: D9136.id21704.diff
Wed, Aug 6, 11:39 AM
F18089521: D9136.id21723.diff
Wed, Aug 6, 11:39 AM
Unknown Object (File)
May 26 2025, 2:50 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