Page MenuHomePhabricator

Clean up Phame Preview
ClosedPublic

Authored by chad on Dec 2 2015, 9:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 16, 1:20 AM
Unknown Object (File)
Fri, Mar 8, 12:48 PM
Unknown Object (File)
Thu, Mar 7, 4:46 AM
Unknown Object (File)
Thu, Mar 7, 4:46 AM
Unknown Object (File)
Sun, Mar 3, 4:30 PM
Unknown Object (File)
Feb 12 2024, 9:05 PM
Unknown Object (File)
Feb 12 2024, 9:03 PM
Unknown Object (File)
Feb 12 2024, 5:46 PM
Subscribers

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPdd82cd49229a: Clean up Phame Preview
Summary

This adds a separate Publish/Unpublish step aside from Preview in Phame Posts. This allows easier access to publishing without previewing, though I left publish in tact on the preview page. Also cleaned up some minor transaction issues with mail.

Test Plan

New Post, Publish Post, Preview Post. Check mail logs. Get mail upon publish.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Clean up Phame Preview.
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/controller/post/PhamePostPublishController.php
40–42

Or $this->newDialog().

49

Or just return $dialog.

This revision is now accepted and ready to land.Dec 2 2015, 9:26 PM
This revision was automatically updated to reflect the committed changes.
chad marked an inline comment as done.