Page MenuHomePhabricator
Differential D8524

Make PhabricatorBotFeedNotificationHandler work with new PhabricatorApplicationTransactionFeedStory
ClosedPublic
Actions

Authored by dctrwatson on Mar 14 2014, 12:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 6, 7:53 PM
Unknown Object (File)
Wed, Dec 4, 10:22 PM
Unknown Object (File)
Sat, Nov 30, 10:43 PM
Unknown Object (File)
Wed, Nov 27, 12:31 AM
Unknown Object (File)
Tue, Nov 26, 4:27 PM
Unknown Object (File)
Tue, Nov 26, 4:27 PM
Unknown Object (File)
Tue, Nov 26, 4:27 PM
Unknown Object (File)
Tue, Nov 26, 4:27 PM
View All Files
Subscribers
aran
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rPa5f6e19dfd66: Make PhabricatorBotFeedNotificationHandler work with new…
Summary

Depends on D8521

Test Plan

ran with handler enabled and phabot posted stories

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dctrwatson updated this revision to Diff 20221.Mar 14 2014, 12:46 AM
dctrwatson retitled this revision from to Make PhabricatorBotFeedNotificationHandler work with new PhabricatorApplicationTransactionFeedStory.
dctrwatson updated this object.
dctrwatson edited the test plan for this revision. (Show Details)
dctrwatson added a reviewer: epriestley.
dctrwatson added a parent revision: D8521: Include objectPHID in feed.query text view.
Herald added a reviewer: Blessed Reviewers.Mar 14 2014, 12:46 AM
Herald added subscribers: Korvin, epriestley, aran.
epriestley requested changes to this revision.Mar 14 2014, 12:58 AM
epriestley edited edge metadata.

Minor nits.

src/infrastructure/daemon/bot/handler/PhabricatorBotFeedNotificationHandler.php
14

For consistency, prefer to spell "Uri" as "URI".

23

Prefer phid_get_type().

157

Prefer phid_get_type() here too.

This revision now requires changes to proceed.Mar 14 2014, 12:58 AM
dctrwatson updated this revision to Diff 20227.Mar 14 2014, 1:04 AM
dctrwatson edited edge metadata.
dctrwatson updated this revision to Diff 20229.Mar 14 2014, 1:15 AM
dctrwatson edited edge metadata.
  • Add 'abandoned'
dctrwatson updated this revision to Diff 20231.Mar 14 2014, 1:18 AM
  • Update usage of typesNeedURI
epriestley accepted this revision.Mar 14 2014, 1:19 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Mar 14 2014, 1:19 AM
epriestley closed this revision.Mar 14 2014, 1:20 AM
epriestley updated this revision to Diff 20232.

Closed by commit rPa5f6e19dfd66 (authored by @dctrwatson, committed by @epriestley).

Revision Contents
Changeset List

PathSize
src/
infrastructure/
daemon/
bot/
handler/
22 lines

Diff 20232

View Options

src/infrastructure/daemon/bot/handler/PhabricatorBotFeedNotificationHandler.php

Loading...
Phabricator · Built by Phacility