Differential D17523

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

Authored by chad on Mar 21 2017, 12:31 AM.

Details

Reviewers

epriestley

Maniphest Tasks

T5307: UX for taking actions with search results

Commits

rP1a5d92184cb6: Try to guess a name for the 'Add to Dashboard' workflow

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

chad created this revision.Mar 21 2017, 12:31 AM

Herald added a subscriber: Korvin. · View Herald TranscriptMar 21 2017, 12:31 AM

Harbormaster completed remote builds in B16062: Diff 42147.Mar 21 2017, 12:32 AM

epriestley accepted this revision.Mar 21 2017, 12:38 AM

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

derp

Harbormaster completed remote builds in B16063: Diff 42148.Mar 21 2017, 12:45 AM

Closed by commit rP1a5d92184cb6: Try to guess a name for the 'Add to Dashboard' workflow (authored by chad, committed by chad). · Explain WhyMar 21 2017, 1:02 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

dashboard/

controller/

PhabricatorDashboardQueryPanelInstallController.php

5 lines

Diff 42149

View Options

src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php

Try to guess a name for the 'Add to Dashboard' workflowClosedPublicActions