Page MenuHomePhabricator

Add Dashboards as a default pinned application
ClosedPublic

Authored by chad on Mar 21 2017, 5:41 PM.
Tags
None
Referenced Files
F18850925: D17527.id42159.diff
Thu, Oct 30, 8:34 PM
F18733521: D17527.id.diff
Sep 30 2025, 10:36 PM
F18677383: D17527.diff
Sep 25 2025, 9:02 PM
F18618060: D17527.id42159.diff
Sep 14 2025, 11:04 PM
F18508660: D17527.id.diff
Sep 5 2025, 2:57 AM
F18500743: D17527.diff
Sep 4 2025, 9:18 PM
F18467041: D17527.id42156.diff
Sep 2 2025, 11:31 AM
F18465146: D17527.diff
Sep 2 2025, 6:32 AM
Subscribers

Details

Summary

Ref T10390. Dashboard usability is high enough that I think we should pin it by default for users to create custom home pages.

Test Plan

Review order of applications in sandbox.

Diff Detail

Repository
rP Phabricator
Branch
install-dash-app (branched from master)
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/dashboard/application/PhabricatorDashboardApplication.php:23XHP62Class Name Literal
Unit
Tests Passed
Build Status
Buildable 16071
Build 21318: Run Core Tests
Build 21317: arc lint + arc unit

Event Timeline

This feels like a fairly advanced application for normal users to me and I think setup/NUX already guides admins to it, but I don't feel strongly.

src/applications/dashboard/application/PhabricatorDashboardApplication.php
22

I think you can just return true;, we won't pin un-installed applications.

This revision is now accepted and ready to land.Mar 21 2017, 6:02 PM
This revision was automatically updated to reflect the committed changes.