Add more information to Dashboard ApplicationSearch list

Authored by chad on Feb 12 2017, 3:49 AM.



Fixes T4984. This is about as fancy as I want to get this pass. Adds in the list of panel titles and the author. This does give me a rough idea what's on each dashboard.

Test Plan

Visit a list of dashboards and see various authors and panels.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
chad created this revision.Feb 12 2017, 3:49 AM
chad edited the test plan for this revision. (Show Details)Feb 12 2017, 3:52 AM
epriestley accepted this revision.Feb 13 2017, 2:30 PM

Ah, nice. This feels like a pretty reasonable compromise between the current "no clue what any of these things are" state and whatever theoretical future "tiny picture of the dashboard which somehow requires 2000 lines of JS" state we might eventually want to build.

(I think in T4984 I was somehow imagining that we'd list the panel types ("Tab Panel, Tab Panel, Query Panel"), not the names, which is why I wasn't too warm on the idea.)

This revision is now accepted and ready to land.Feb 13 2017, 2:30 PM
This revision was automatically updated to reflect the committed changes.