Differential D14619

Allow builtin Phame UI to be publicly viewable
ClosedPublic
Actions

Authored by chad on Dec 1 2015, 12:53 AM.

Details

Reviewers

epriestley

Maniphest Tasks

T9360: Unbeta Phame

Commits

Restricted Diffusion Commit
rPb2b652ef288e: Allow builtin Phame UI to be publicly viewable

Summary

These weren't open to the public even if the blog was public.

Test Plan

View a blog post, blog view, and blog manage page while logged out.

Diff Detail

Repository

rP Phabricator

Branch

phame-blog-edit

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 9175
Build 10844: Run Core Tests
Build 10843: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 35365.Dec 1 2015, 12:53 AM

chad retitled this revision from to Allow builtin Phame UI to be publicly viewable.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

Herald added a subscriber: Korvin. · View Herald TranscriptDec 1 2015, 12:53 AM

epriestley accepted this revision.Dec 1 2015, 12:53 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 1 2015, 12:53 AM

Closed by commit rPb2b652ef288e: Allow builtin Phame UI to be publicly viewable (authored by chad, committed by chad). · Explain WhyDec 1 2015, 12:54 AM

This revision was automatically updated to reflect the committed changes.

chad added a task: T9360: Unbeta Phame.Dec 1 2015, 1:06 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

phame/

controller/

blog/

PhameBlogManageController.php

4 lines

PhameBlogViewController.php

8 lines

Diff 35365

View Options

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