Page MenuHomePhabricator

Add "Installed" icon to Dashboard list view.
ClosedPublic

Authored by btrahan on Aug 7 2014, 8:34 PM.
Tags
None
Referenced Files
F18816334: D10181.diff
Tue, Oct 21, 6:07 AM
F18796874: D10181.id.diff
Fri, Oct 17, 4:19 AM
F18791281: D10181.diff
Thu, Oct 16, 8:53 AM
F18627645: D10181.id.diff
Sep 16 2025, 4:27 AM
F18621562: D10181.diff
Sep 15 2025, 8:46 AM
F18616419: D10181.diff
Sep 14 2025, 5:28 PM
F17852481: D10181.id24492.diff
Jul 27 2025, 12:41 PM
F17843658: D10181.diff
Jul 27 2025, 1:12 AM
Subscribers

Details

Summary

Fixes T5478. For "personal" installs use the person icon; for global use the global icon. For both providing explanatory tooltip text about what's going on. This will need to be updated if / when we start installing dashboards to other applications. Also, this query isn't 100% optimized but the major part *is* so I think its okay.

Test Plan

Installed a dashboard for personal use and verified correct icon / text showed up. Did the same for global installed dashboard...!

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan retitled this revision from to Add "Installed" icon to Dashboard list view..
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.

Seems reasonable to me.

This revision is now accepted and ready to land.Aug 7 2014, 8:43 PM
btrahan updated this revision to Diff 24492.

Closed by commit rP7388351aab9c (authored by @btrahan).