Page MenuHomePhabricator

Add a transaction for PhamePost visibility
ClosedPublic

Authored by chad on Nov 7 2015, 4:10 AM.
Tags
None
Referenced Files
F15463920: D14429.id34867.diff
Wed, Apr 2, 4:30 AM
F15454221: D14429.id34864.diff
Sat, Mar 29, 5:03 PM
F15452065: D14429.id34865.diff
Sat, Mar 29, 1:29 AM
F15448565: D14429.id.diff
Fri, Mar 28, 5:38 AM
F15443425: D14429.diff
Thu, Mar 27, 4:19 AM
F15406889: D14429.id34864.diff
Tue, Mar 18, 2:32 PM
F15402363: D14429.id34865.diff
Mon, Mar 17, 10:09 PM
F15348598: D14429.diff
Mar 10 2025, 2:35 PM
Subscribers

Details

Reviewers
epriestley
Maniphest Tasks
T9360: Unbeta Phame
Commits
Restricted Diffusion Commit
rPf8b085c574ae: Add a transaction for PhamePost visibility
Summary

Adds ability to set visibility when authoring a Post. New default is "Visible". If you write a post and save it as a Draft, and later click publish, a feed story and mail will go out.

Test Plan

Write a new Post, see feed story and get email. Write a new Draft, get nothing. Click Publish, see story and email.

Diff Detail

Repository
rP Phabricator
Branch
phame-visibility
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8670
Build 10051: Run Core Tests
Build 10050: arc lint + arc unit

Event Timeline

chad retitled this revision from to Add a transaction for PhamePost visibility.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/phame/storage/PhamePostTransaction.php
219

if (strlen($text)) is a slightly better check here, so we summarize post bodies like "0" and "0.0" correctly.

This revision is now accepted and ready to land.Nov 7 2015, 12:24 PM

(I think this application behavior is a good one, too.)

This revision was automatically updated to reflect the committed changes.