Page MenuHomePhabricator
Differential D21613

Remove support for "paths" parameter in "differential.query"
ClosedPublic
Actions

Authored by epriestley on Mar 15 2021, 4:38 PM.
Tags
None
Referenced Files
F19938780: D21613.diff
Tue, Apr 14, 6:59 AM
F19920902: D21613.id.diff
Mon, Apr 6, 12:44 AM
F19914264: D21613.id51459.diff
Fri, Apr 3, 9:11 AM
F19914262: D21613.id51459.diff
Fri, Apr 3, 9:11 AM
F19913331: D21613.id.diff
Fri, Apr 3, 1:31 AM
F19880910: D21613.diff
Wed, Mar 18, 9:11 PM
F19833216: D21613.id51459.diff
Mar 11 2026, 1:06 AM
F19785872: D21613.id51459.diff
Feb 25 2026, 1:58 AM
View All Files
Subscribers
None

Details

Reviewers
None
Commits
rPbcd592cf7e15: Remove support for "paths" parameter in "differential.query"
Summary

See T13639. This change simplifies providing a more modern approach to querying this data via "differential.revision.search".

Test Plan

Called "differential.query" with paths (got an error) and without paths (got a valid query result).

Diff Detail

Repository
rP Phabricator
Branch
recent2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25257
Build 34860: Run Core Tests
Build 34859: arc lint + arc unit

Event Timeline

epriestley created this revision.Mar 15 2021, 4:38 PM
Harbormaster completed remote builds in B25257: Diff 51449.Mar 15 2021, 4:39 PM
epriestley requested review of this revision.Mar 15 2021, 4:39 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 15 2021, 11:16 PM
Closed by commit rPbcd592cf7e15: Remove support for "paths" parameter in "differential.query" (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPbcd592cf7e15: Remove support for "paths" parameter in "differential.query".

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
conduit/
49 lines

Diff 51449

View Options

src/applications/differential/conduit/DifferentialQueryConduitAPIMethod.php

Loading...
Phabricator · Built by Phacility