Page MenuHomePhabricator

Add the skeleton for a "transaction.search" Conduit API method
ClosedPublic

Authored by epriestley on Aug 24 2017, 9:20 PM.
Tags
None
Referenced Files
F19827841: D18466.id44369.diff
Mon, Mar 9, 1:23 PM
F19811976: D18466.id44369.diff
Wed, Mar 4, 3:12 AM
F19810588: D18466.diff
Tue, Mar 3, 8:34 PM
F19795600: D18466.id.diff
Thu, Feb 26, 11:33 PM
F19784520: D18466.diff
Tue, Feb 24, 8:18 PM
F19672428: D18466.id44369.diff
Sun, Feb 8, 3:16 AM
F19509911: D18466.id44369.diff
Jan 11 2026, 2:49 AM
F19231899: D18466.diff
Dec 21 2025, 4:56 PM
Subscribers
None

Details

Summary

Ref T5873. See PHI14. This does the basics that are shared across everything (IDs, PHIDs, dates, comments).

It doesn't do types (I think I don't necessarily want to expose internal types over the API?) or transaction-specific data.

In the next change, I'm going to add ways to let ModularTransactions "opt-in" to providing more data to Conduit. I'll use this to flesh out the actual desired transaction types (comments, presumably inline comments) and likely leave the rest as skeletons for now until use cases arise so we don't create a backward compatibility issue (or a security issue!) by exposing tons of internal stuff as public-facing API.

Test Plan

Ran queries, used paging. Retrieved an edited, deleted, and normal comment.

Screen Shot 2017-08-24 at 2.14.41 PM.png (1×1 px, 330 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable