Add more information to Dashboard ApplicationSearch list
ClosedPublic

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

Details

Summary

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

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
chad created this revision.Sun, Feb 12, 3:49 AM
chad edited the test plan for this revision. (Show Details)Sun, Feb 12, 3:52 AM
epriestley accepted this revision.Mon, Feb 13, 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.Mon, Feb 13, 2:30 PM
This revision was automatically updated to reflect the committed changes.