Page MenuHomePhabricator

Add Herald Adapters to Phame
ClosedPublic

Authored by chad on Dec 28 2015, 2:05 AM.
Tags
None
Referenced Files
F14473765: D14897.diff
Fri, Dec 27, 3:09 PM
Unknown Object (File)
Thu, Dec 26, 7:15 PM
Unknown Object (File)
Thu, Dec 19, 4:45 PM
Unknown Object (File)
Sat, Dec 14, 9:19 PM
Unknown Object (File)
Fri, Dec 13, 8:30 PM
Unknown Object (File)
Mon, Dec 9, 2:10 PM
Unknown Object (File)
Tue, Dec 3, 11:41 PM
Unknown Object (File)
Sun, Dec 1, 7:27 PM
Subscribers

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rP744215d5ffe4: Add Herald Adapters to Phame
Summary

Adds a basic HeraldAdapter to Phame Blogs and Posts.

Test Plan

Make a Herald rule to CC me on new posts or blogs automatically.

Diff Detail

Repository
rP Phabricator
Branch
phame-herald (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 9832
Build 11835: Run Core Tests
Build 11834: arc lint + arc unit

Event Timeline

chad retitled this revision from to Add Herald Adapters to Phame.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
src/applications/phame/herald/HeraldPhameBlogAdapter.php
60

I made these up?

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

The getHeraldName() stuff just shows up on the "Transcripts" view, I think, and should be human-readable.

This revision is now accepted and ready to land.Jan 1 2016, 11:39 AM
chad edited edge metadata.
This revision was automatically updated to reflect the committed changes.