Page MenuHomePhabricator
Differential D9235

Allow a limit to be set on the number of results in a query panel
ClosedPublic
Actions

Authored by hach-que on May 21 2014, 7:10 AM.
Tags
None
Referenced Files
F18808876: D9235.diff
Sun, Oct 19, 9:24 AM
F18760639: D9235.id.diff
Mon, Oct 6, 9:57 AM
F18756221: D9235.diff
Sun, Oct 5, 10:12 AM
F18667947: D9235.id21934.diff
Sep 24 2025, 12:58 PM
F18557747: D9235.id.diff
Sep 8 2025, 8:05 PM
F18339802: D9235.id21930.diff
Aug 25 2025, 9:45 PM
F18111495: D9235.id.diff
Aug 12 2025, 11:05 AM
F18090152: D9235.id21930.diff
Aug 6 2025, 2:29 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T4980: Move Dashboards to beta
Commits
Restricted Diffusion Commit
rP49c15a6d951c: Allow a limit to be set on the number of results in a query panel
Summary

This allows a maximum number of items to be set in a query panel. Mostly useful when you have a query panel on the feed search and you don't want 4 billion results cluttering your dashboard.

Test Plan

Created a query panel with a maximum and it worked. Left it blank and got the default results.

Diff Detail

Repository
rP Phabricator
Branch
dashboard-query-limit
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 624
Build 624: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

hach-que updated this revision to Diff 21930.May 21 2014, 7:10 AM
hach-que retitled this revision from to Allow a limit to be set on the number of results in a query panel.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que added a task: T4980: Move Dashboards to beta.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 21 2014, 7:10 AM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B624: Diff 21930.May 21 2014, 7:12 AM
epriestley accepted this revision.May 21 2014, 5:09 PM
epriestley edited edge metadata.

We should probably default this to, like, 10, too.

This revision is now accepted and ready to land.May 21 2014, 5:09 PM
epriestley closed this revision.May 21 2014, 5:09 PM
epriestley updated this revision to Diff 21934.
epriestley added a subscriber: Unknown Object (User).

Closed by commit rP49c15a6d951c (authored by @jamesr, committed by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
dashboard/
paneltype/
13 lines

Diff 21930

View Options

src/applications/dashboard/paneltype/PhabricatorDashboardPanelTypeQuery.php

Loading...
Phabricator · Built by Phacility