Page MenuHomePhabricator

Add a Dashboard MenuItem
ClosedPublic

Authored by chad on Dec 16 2016, 9:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 18, 12:15 PM
Unknown Object (File)
Jan 31 2025, 5:05 PM
Unknown Object (File)
Jan 29 2025, 8:30 AM
Unknown Object (File)
Jan 4 2025, 8:33 AM
Unknown Object (File)
Dec 19 2024, 11:10 AM
Unknown Object (File)
Dec 19 2024, 7:44 AM
Unknown Object (File)
Dec 11 2024, 3:53 AM
Unknown Object (File)
Dec 10 2024, 6:37 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.