Page MenuHomePhabricator

Add a Dashboard MenuItem
ClosedPublic

Authored by chad on Dec 16 2016, 9:03 PM.
Tags
None
Referenced Files
F18780611: D17092.id41111.diff
Sun, Oct 12, 5:51 PM
F18625847: D17092.diff
Sep 15 2025, 11:44 PM
F18363744: D17092.id.diff
Aug 27 2025, 5:16 PM
F18350513: D17092.diff
Aug 26 2025, 7:53 PM
F18087597: D17092.id41109.diff
Aug 6 2025, 4:37 AM
F18084505: D17092.diff
Aug 5 2025, 3:50 PM
F17839434: D17092.id.diff
Jul 26 2025, 7:57 PM
F17809152: D17092.diff
Jul 25 2025, 4:35 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.