Page MenuHomePhabricator

Add mail support to PhameBlog
ClosedPublic

Authored by chad on Nov 7 2015, 8:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 11, 4:18 PM
Unknown Object (File)
Sun, Dec 8, 8:50 PM
Unknown Object (File)
Sun, Dec 8, 3:42 PM
Unknown Object (File)
Sat, Dec 7, 1:38 AM
Unknown Object (File)
Thu, Dec 5, 5:09 PM
Unknown Object (File)
Tue, Dec 3, 5:36 PM
Unknown Object (File)
Tue, Dec 3, 4:11 PM
Unknown Object (File)
Fri, Nov 29, 3:20 AM
Subscribers

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPc3ecea9788c1: Add mail support to PhameBlog
Summary

Add some mailkeys, allow feed stories to be published.

Test Plan

New Blog, Edit Blog

Diff Detail

Repository
rP Phabricator
Branch
phame-blog-mailkey
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8688
Build 10078: Run Core Tests
Build 10077: arc lint + arc unit

Event Timeline

chad retitled this revision from to Add mail support to PhameBlog.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
  • correct subscribers spelling

Sadly have not gotten this to work. Error is as follows, but I can't find a reason for it to trigger.

Unable to load query for transaction type "BLOG"! at [<phabricator>/src/applications/transactions/worker/PhabricatorApplicationTransactionPublishWorker.php:133]

That error likely means you need to add a new PhameBlogTransactionQuery class. You can look at PhamePostTransactionQuery for an example -- it's pretty straightforward.

epriestley edited edge metadata.

Rest of this looks fine to me.

This revision now requires changes to proceed.Nov 8 2015, 2:25 PM
chad edited edge metadata.
  • correct subscribers spelling
  • Fix remaining mail/feed bugs
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 8 2015, 4:09 PM
This revision was automatically updated to reflect the committed changes.