Page MenuHomePhabricator

Enrich "draft", "summary", and "testPlan" transactions from Differential in "transaction.search"
ClosedPublic

Authored by epriestley on May 9 2019, 4:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 23, 12:43 AM
Unknown Object (File)
Sat, Mar 23, 12:43 AM
Unknown Object (File)
Sat, Mar 23, 12:43 AM
Unknown Object (File)
Sat, Mar 23, 12:43 AM
Unknown Object (File)
Sat, Mar 16, 3:42 AM
Unknown Object (File)
Feb 16 2024, 2:59 PM
Unknown Object (File)
Jan 27 2024, 3:17 PM
Unknown Object (File)
Dec 28 2023, 4:59 PM
Subscribers
None

Details

Reviewers
amckinley
Summary

See PHI1232, which describes a reasonable use case for wanting information about the "draft" ("Hold as Draft / Do Not Auto-Promote") flag.

Also, flesh out "testPlan" and "summary". It's possible these "blob of remarkup" fields might have metadata some day (e.g., a rendered version or a list of PHIDs or something), but we could add more keys, and we already have some other transactions which work like this.

Test Plan

Used "transaction.search" to fetch these transaction types, saw type information and metadata.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

amckinley added inline comments.
src/applications/differential/xaction/DifferentialRevisionTitleTransaction.php
62

Does this (and the change in DifferentialRevisionUpdateTransaction) reflect an API change that happened at some point, or were these arguments just named incorrectly from the beginning?

This revision is now accepted and ready to land.May 16 2019, 12:59 PM

I only blamed around a little bit, but I'm nearly certain they were named incorrectly from the beginning.

I think they got introduced at the same time as ConduitResultInterface, which looks somewhat similar but does have an $object parameter, and a transaction is technically an $object so this just escaped notice.

In either case, not an API change, just a nomenclature change.