Page MenuHomePhabricator

Add a Dashboard MenuItem
ClosedPublic

Authored by chad on Dec 16 2016, 9:03 PM.
Tags
None
Referenced Files
F15474011: D17092.diff
Sun, Apr 6, 4:01 AM
F15419064: D17092.id41109.diff
Fri, Mar 21, 3:09 AM
F15417632: D17092.diff
Thu, Mar 20, 5:42 PM
F15378087: D17092.id41111.diff
Thu, Mar 13, 12:33 PM
F15368411: D17092.id41111.diff
Wed, Mar 12, 1:55 AM
F15344737: D17092.id41111.diff
Mon, Mar 10, 6:33 AM
Unknown Object (File)
Feb 18 2025, 12:15 PM
Unknown Object (File)
Jan 31 2025, 5:05 PM
Subscribers

Details

Summary

Built similar to Projects, allows setting of a Dashboard to MenuItem.

Test Plan

Add a dashboard with and without a name / icon to a Project.

Diff Detail

Repository
rP Phabricator
Branch
dashboard-menuitem (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14998
Build 19667: Run Core Tests
Build 19666: arc lint + arc unit

Event Timeline

This works really well, if it ajaxed in, we'd have Project Dashboards.

epriestley edited edge metadata.

In both this and projects, I wonder if we should flip the "name" and "thing" fields, so they primary/required field is on top? Although maybe you're usually going to want to rename dashboards.

This revision is now accepted and ready to land.Dec 16 2016, 9:13 PM

I could maybe call the field "Name (optional):"

You could setPlaceHolder(pht('This is an optional field which you do not need to fill out if you would prefer not to fill it out')) too, I think.

This revision was automatically updated to reflect the committed changes.