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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.