Paths

Table of Contentst

Completely remove "differential.find" Conduit API method
ClosedPublic
Actions

Authored by epriestley on Aug 11 2017, 4:34 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Nov 12, 4:15 PM

	Unknown Object (File)
	Mon, Oct 28, 10:02 PM

	Unknown Object (File)
	Wed, Oct 23, 12:16 PM

	Unknown Object (File)
	Oct 2 2024, 9:05 AM

	Unknown Object (File)
	Sep 11 2024, 10:13 AM

	Unknown Object (File)
	Sep 9 2024, 5:09 PM

	Unknown Object (File)
	Aug 19 2024, 7:16 AM

	Unknown Object (File)
	Aug 16 2024, 11:18 PM

View All Files

Subscribers

None

Tokens

"Burninate" token, awarded by chad.

Details

Reviewers

chad

Maniphest Tasks

T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential

Commits

rP42020e135767: Completely remove "differential.find" Conduit API method

Summary

Ref T2543. I believe there have been no upstream callsites of this method since D1646, in February 2012.

The method works, and we can revert this if needbe, but this seems like a good time to remove support.

Test Plan

Grepped for differential.find.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 11 2017, 4:34 PM

Harbormaster completed remote builds in B18010: Diff 44212.Aug 11 2017, 4:35 PM

chad accepted this revision.Aug 11 2017, 5:14 PM

This revision is now accepted and ready to land.Aug 11 2017, 5:14 PM

chad awarded a token.Aug 11 2017, 5:15 PM

Closed by commit rP42020e135767: Completely remove "differential.find" Conduit API method (authored by epriestley, committed by epriestley). · Explain WhyAug 11 2017, 8:38 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

applications/

differential/

conduit/

DifferentialFindConduitAPIMethod.php

Diff 44239

View Options

src/__phutil_library_map__.php