Page MenuHomePhabricator

Try to guess a name for the 'Add to Dashboard' workflow
ClosedPublic

Authored by chad on Mar 21 2017, 12:31 AM.
Tags
None
Referenced Files
F18805818: D17523.id42148.diff
Sat, Oct 18, 4:38 PM
F18766633: D17523.id42148.diff
Tue, Oct 7, 5:39 PM
F18756732: D17523.id.diff
Sun, Oct 5, 1:08 PM
F18751898: D17523.diff
Sat, Oct 4, 1:10 PM
F18751186: D17523.diff
Sat, Oct 4, 9:58 AM
F18746122: D17523.id42147.diff
Fri, Oct 3, 11:23 AM
F18629280: D17523.id42147.diff
Sep 16 2025, 8:25 AM
F18628611: D17523.id.diff
Sep 16 2025, 6:43 AM
Subscribers

Details

Summary

Ref T5307. Just makes the dialog a little easier to use. Picks a name if we already have one.

Test Plan

Test a builtin, custom saved, and a new advanced search (no name).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley added inline comments.
src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php
47–52

You can just do the load bit here, right? There are two branches and we're running the same code on both?

This revision is now accepted and ready to land.Mar 21 2017, 12:38 AM
This revision was automatically updated to reflect the committed changes.