Page MenuHomePhabricator

Add a Dashboard MenuItem
ClosedPublic

Authored by chad on Dec 16 2016, 9:03 PM.
Tags
None
Referenced Files
F17839434: D17092.id.diff
Sat, Jul 26, 7:57 PM
F17809152: D17092.diff
Fri, Jul 25, 4:35 PM
Unknown Object (File)
Jun 25 2025, 10:39 PM
Unknown Object (File)
Jun 20 2025, 9:05 AM
Unknown Object (File)
Jun 18 2025, 5:58 PM
Unknown Object (File)
Jun 17 2025, 8:52 PM
Unknown Object (File)
Jun 6 2025, 2:50 PM
Unknown Object (File)
May 29 2025, 8:54 AM
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.