Page MenuHomePhabricator

Add a transaction for PhamePost visibility
ClosedPublic

Authored by chad on Nov 7 2015, 4:10 AM.
Tags
None
Referenced Files
F19341274: D14429.id34864.diff
Thu, Dec 25, 11:09 PM
F19087454: D14429.id34865.diff
Wed, Dec 3, 12:42 AM
F19087453: D14429.id34864.diff
Wed, Dec 3, 12:42 AM
F19087193: D14429.diff
Wed, Dec 3, 12:00 AM
F19058599: D14429.diff
Sat, Nov 29, 2:52 AM
F18848293: D14429.id.diff
Oct 30 2025, 4:14 AM
F18844082: D14429.diff
Oct 29 2025, 3:01 AM
F18807940: D14429.id34867.diff
Oct 19 2025, 4:02 AM
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.