Paths

Table of Contentst

Differential D10063

(Partial) Provide more useful information on feed.http-hooks
AbandonedPublic
Actions

Authored by epriestley on Jul 28 2014, 4:35 PM.

Tags

None

Referenced Files

	F15507642: D10063.diff
	Tue, Apr 15, 9:52 PM

	F15459031: D10063.id24195.diff
	Mon, Mar 31, 9:13 AM

	F15456093: D10063.diff
	Sun, Mar 30, 7:00 AM

	Unknown Object (File)
	Feb 22 2025, 7:29 AM

	Unknown Object (File)
	Feb 7 2025, 10:00 PM

	Unknown Object (File)
	Feb 7 2025, 10:00 PM

	Unknown Object (File)
	Jan 29 2025, 10:53 AM

	Unknown Object (File)
	Jan 28 2025, 7:05 AM

Subscribers

View All 9 Subscribers

Details

Reviewers

Maniphest Tasks

T5726: Add more information to story POSTs during the feed.http-hooks hook

Summary

Fixes T5726.

Test Plan

Not yet.

Diff Detail

Repository

Branch

morehookdata

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1913
Build 1914: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24195.Jul 28 2014, 4:35 PM

epriestley retitled this revision from to (Partial) Provide more useful information on feed.http-hooks.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a task: T5726: Add more information to story POSTs during the feed.http-hooks hook.

Herald added a subscriber: epriestley. · View Herald TranscriptJul 28 2014, 4:35 PM

Harbormaster completed remote builds in B1913: Diff 24195.Jul 28 2014, 4:36 PM

@epriestley, would it be possible to merge this guy soon? I'm finding my feed.http-hooks stories kind of information-sparse :)

• chasemp added a subscriber: • chasemp.Oct 6 2014, 2:09 AM

epriestley mentioned this in T5726: Add more information to story POSTs during the feed.http-hooks hook.Jan 5 2015, 11:43 PM

Bump, is there a particular reason for a hold up on this?

joshuaspence added a subscriber: joshuaspence.Feb 27 2015, 7:52 AM

Bump; I'd like to see this added as well. Additionally, is there a way to get the actual comment text from "new comment" feed items? As far as I can see that is not possible now, and is not added in this revision.

• trigged added a subscriber: • trigged.Jun 9 2015, 7:10 AM

bump?

• jverkoey added a subscriber: • jverkoey.Nov 17 2015, 2:47 AM

chad added a reviewer: chad.Aug 21 2016, 8:02 PM

chad resigned from this revision.Apr 13 2017, 8:03 PM

Obsoleted by webhook support.

Revision Contents
Changeset List

Path

Size

src/

applications/

feed/

controller/

PhabricatorFeedDetailController.php

9 lines

worker/

FeedPublisherHTTPWorker.php

37 lines

Diff 24195

src/applications/feed/controller/PhabricatorFeedDetailController.php

Loading...

src/applications/feed/worker/FeedPublisherHTTPWorker.php

Loading...