Differential D9015

Move dashboard list rendering to ApplicationSearch
ClosedPublic
Actions

Authored by epriestley on May 8 2014, 4:32 PM.

Details

Reviewers

btrahan

Maniphest Tasks

T4986: Implement new Dashboard panel types

Commits

Restricted Diffusion Commit
rPc6a68aadc08f: Move dashboard list rendering to ApplicationSearch

Summary

Ref T4986. This is "good" and "desirable".

Test Plan

Saw dashboard list, panel.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 21404.May 8 2014, 4:32 PM

epriestley retitled this revision from to Move dashboard list rendering to ApplicationSearch.

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 8 2014, 4:32 PM

Harbormaster completed remote builds in B306: Diff 21404.May 8 2014, 4:33 PM

btrahan accepted this revision.May 8 2014, 7:06 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.May 8 2014, 7:06 PM

Closed by commit rPc6a68aadc08f (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

dashboard/

controller/

PhabricatorDashboardListController.php

26 lines

query/

PhabricatorDashboardSearchEngine.php

29 lines

Diff 21429

View Options

src/applications/dashboard/controller/PhabricatorDashboardListController.php