Homec4science

Include storyText in feed POST data

Authored by Ben Alpert <spicyjalapeno@gmail.com> on Apr 6 2014, 20:48.

Description

Include storyText in feed POST data

Summary: I also changed PhabricatorApplicationTransactionFeedStory and the TokenGivenFeedStory to include only the title/first line of the feed story, which is more convenient (previously, strip_tags gave a multi-line story without even any linebreaks) and more consistent with the other story types.

Test Plan: Added a requestbin URL to feed.http-hooks, commented on a Differential, and saw storyText equal to "alpert added a comment to D2: c." in the POST data it received.

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

Maniphest Tasks: T4630

Differential Revision: https://secure.phabricator.com/D8710

Details

Committed
epriestley <git@epriestley.com>Apr 6 2014, 20:48
Pushed
aubortJan 31 2017, 17:16
Parents
rPHb03912213f52: Add original mail headers to the bottom of error mail
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHcd829434d49d: Include storyText in feed POST data (authored by Ben Alpert <spicyjalapeno@gmail.com>).Apr 6 2014, 20:48