Page MenuHomePhabricator
Differential D17520

Add better error checking to 'Add to Dashboard'
ClosedPublic
Actions

Authored by chad on Mar 20 2017, 9:47 PM.
Tags
None
Referenced Files
F13282848: D17520.diff
Sun, Jun 2, 12:30 PM
F13268478: D17520.diff
Wed, May 29, 5:05 AM
F13256723: D17520.id.diff
Sat, May 25, 4:27 PM
F13255412: D17520.id.diff
Sat, May 25, 4:48 AM
F13254012: D17520.diff
Sat, May 25, 3:06 AM
F13250858: D17520.id42132.diff
Fri, May 24, 6:02 PM
F13250814: D17520.id42131.diff
Fri, May 24, 5:44 PM
F13233013: D17520.diff
Tue, May 21, 1:38 AM
View All 96 Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T5307: UX for taking actions with search results
Commits
rP9b07adb8dad3: Add better error checking to 'Add to Dashboard'
Summary

Ref T5307. Adds a better query check query, sets required for the name, adds the correct URI for cancelling.

Test Plan

Test a form without a name, fake a query string, test cancel button.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.Mar 20 2017, 9:47 PM
Herald added a subscriber: Korvin. · View Herald TranscriptMar 20 2017, 9:47 PM
Harbormaster completed remote builds in B16046: Diff 42131.Mar 20 2017, 9:48 PM
epriestley accepted this revision.Mar 20 2017, 9:52 PM
epriestley added inline comments.
src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php
44

who's a good query

is it you? are you a good query??

pet pet pat yes you're a good query

This revision is now accepted and ready to land.Mar 20 2017, 9:52 PM
chad added a comment.Mar 20 2017, 9:55 PM

I'm not a fan of adding the Dashboard ID to select, but maybe I could just switch to typeahead if you have more than n dashboards. I just want to solve a real problem not a test data problem. Will see what happens when I use it on this install. :)

Closed by commit rP9b07adb8dad3: Add better error checking to 'Add to Dashboard' (authored by chad, committed by chad). · Explain WhyMar 20 2017, 9:55 PM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Mar 20 2017, 10:01 PM

404

https://secure.phabricator.com/dashboard/panel/install/DifferentialRevisionSearchEngine/authored/

epriestley added inline comments.Mar 20 2017, 10:06 PM
src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php
32

Oh, this should be $v_query.

Revision Contents
Changeset List

PathSize
src/
applications/
dashboard/
controller/
24 lines

Diff 42132

View Options

src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php

Loading...
Phabricator · Built by Phacility