Page MenuHomePhabricator
Differential D14432

Add Subscribers to Phame Blogs / Posts
ClosedPublic
Actions

Authored by chad on Nov 7 2015, 6:12 PM.
Tags
None
Referenced Files
F19511445: D14432.id.diff
Mon, Jan 12, 2:41 AM
F19176234: D14432.id34898.diff
Mon, Dec 15, 11:46 PM
F19174839: D14432.id34898.diff
Mon, Dec 15, 11:11 PM
F19118245: D14432.diff
Dec 9 2025, 8:14 PM
F19118128: D14432.diff
Dec 9 2025, 8:12 PM
F19093013: D14432.id34874.diff
Dec 3 2025, 7:19 PM
F19093012: D14432.id34873.diff
Dec 3 2025, 7:19 PM
F18842561: D14432.id34898.diff
Oct 28 2025, 4:27 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9051: Add subscribers/CCs to Phame Blog
Commits
Restricted Diffusion Commit
rPc86a514f8478: Add Subscribers to Phame Blogs / Posts
Summary

Fixes T9051, adds ability to edit blogs and posts and manually add subscribers. Also fixed bug granting tokens to posts.

Test Plan

Create a new blog, subcribe chad and notchad. Write a post, both are notified. Award token for hard work.

Diff Detail

Repository
rP Phabricator
Branch
blog-subscribers
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8682
Build 10066: Run Core Tests
Build 10065: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 34873.Nov 7 2015, 6:12 PM
chad retitled this revision from to Add Subscribers to Phame Blogs / Posts.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T9051: Add subscribers/CCs to Phame Blog.
Herald added a subscriber: Korvin. · View Herald TranscriptNov 7 2015, 6:12 PM
chad updated this revision to Diff 34874.Nov 7 2015, 7:30 PM
  • Remove unneeded transactions
epriestley accepted this revision.Nov 8 2015, 2:21 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 8 2015, 2:21 PM
Closed by commit rPc86a514f8478: Add Subscribers to Phame Blogs / Posts (authored by chad, committed by chad). · Explain WhyNov 8 2015, 3:04 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
phame/
controller/
blog/
40 lines
post/
15 lines
editor/
4 lines
8 lines

Diff 34873

View Options

src/applications/phame/controller/blog/PhameBlogEditController.php

Loading...
View Options

src/applications/phame/controller/post/PhamePostEditController.php

Loading...
View Options

src/applications/phame/editor/PhameBlogEditor.php

Loading...
View Options

src/applications/phame/editor/PhamePostEditor.php

Loading...
Phabricator · Built by Phacility