Page MenuHomePhabricator

Add ability to archive a Dashboard
ClosedPublic

Authored by chad on Jul 22 2015, 10:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 6:02 AM
Unknown Object (File)
Fri, Dec 20, 5:47 AM
Unknown Object (File)
Thu, Dec 19, 2:28 PM
Unknown Object (File)
Sun, Dec 15, 7:52 AM
Unknown Object (File)
Sat, Dec 14, 1:12 AM
Unknown Object (File)
Fri, Dec 13, 2:19 PM
Unknown Object (File)
Fri, Dec 13, 2:57 AM
Unknown Object (File)
Fri, Dec 6, 9:09 PM
Subscribers

Details

Summary

Let's people archive unused Dashboards (if they have permission).

Test Plan

Archive and Unarchive a dashboard, view history, view search, build new filters.

Diff Detail

Repository
rP Phabricator
Branch
dashboards-search
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7356
Build 7752: [Placeholder Plan] Wait for 30 Seconds
Build 7751: arc lint + arc unit

Event Timeline

chad retitled this revision from to Add ability to archive a Dashboard.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
epriestley edited edge metadata.

Maybe align OPEN/CLOSED and 'open' / 'closed' better with "Archived" / "Active", since we don't have any legacy cruft here?

src/applications/dashboard/editor/PhabricatorDashboardTransactionEditor.php
70–72

Probably slightly better to set this in initializeNewDashboard().

src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php
25–26

This method is static, so just call it as PhabricatorDashboard::getStatusNameMap().

152

This is named "Manage", but the UI on this screen has a link to "Manage Dashboard", which takes you to the other place. This is more like "view", and is fully redundant with just clicking the link title.

161

This should be disabled if the user doesn't have permission to edit the dhasboard.

This revision is now accepted and ready to land.Jul 23 2015, 8:35 PM
chad marked 4 inline comments as done.
chad edited edge metadata.
  • Updates
src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php
52

ahahahaha

This revision was automatically updated to reflect the committed changes.