Paths

Table of Contentst

Differential D25777

Make maniphest.gettasktransactions include metadata and title
ClosedPublic
Actions

Authored by aklapper on Aug 14 2024, 13:18.

Tags

None

Referenced Files

	F2694562: D25777.1734788795.diff
	Fri, Dec 20, 13:46

	F2693762: D25777.1734769941.diff
	Fri, Dec 20, 08:32

Subscribers

valerio.bozzolan

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15916: Conduit maniphest.gettasktransactions: include metadata and title

Commits

rP6ec5c88bee24: Make maniphest.gettasktransactions include metadata and title

Summary

Include task title and metadata in Conduit's maniphest.gettasktransactions output.
This data can for example be used by IRC bots to improve their output (e.g. conditions which changes [not] to emit or text coloring based on metadata).

Upstreaming from https://phabricator.wikimedia.org/rPHABf2fd14dc1edeb41aa2874336548cfaa7fa0e87a0

NOTE: this specific API is in "frozen" status. This method is frozen and will eventually be deprecated. New code should use "transaction.search" instead.

Closes T15916

Test Plan

Call maniphest.gettasktransactions on a task, see two additional fields.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

aklapper created this revision.Aug 14 2024, 13:18

Owners added a reviewer: O1: Blessed Committers.Aug 14 2024, 13:18

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptAug 14 2024, 13:18

aklapper requested review of this revision.Aug 14 2024, 13:18

Harbormaster completed remote builds in B1527: Diff 2370.Aug 14 2024, 13:18

valerio.bozzolan edited the summary of this revision. (Show Details)Aug 14 2024, 16:02

valerio.bozzolan added a task: T15916: Conduit maniphest.gettasktransactions: include metadata and title.

valerio.bozzolan edited the summary of this revision. (Show Details)Sep 9 2024, 07:33

Wikimedia Foundation has this in production since years. I think it doesn't hurt.

I hope this attracts interest in the related task.

In any case, consider sharing a comment about this:

This revision is now accepted and ready to land.Nov 12 2024, 15:11

Closed by commit rP6ec5c88bee24: Make maniphest.gettasktransactions include metadata and title. · Explain WhyTue, Nov 26, 13:08

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rP6ec5c88bee24: Make maniphest.gettasktransactions include metadata and title.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

maniphest/

conduit/

ManiphestGetTaskTransactionsConduitAPIMethod.php

2 lines

O1: Blessed Committers

Diff 2534

src/applications/maniphest/conduit/ManiphestGetTaskTransactionsConduitAPIMethod.php

Loading...