Allow public on list of subscribers
ClosedPublic
Actions

Authored by chad on Feb 18 2015, 7:07 PM.

Details

Reviewers

epriestley
btrahan

Maniphest Tasks

T7317: Can't view other subscribers while logged out

Commits

Restricted Diffusion Commit
rPf9638edf37a3: Allow public on list of subscribers

Summary

Fixes T7317, allows public to be set on this list controller.

Test Plan

Tested a list of subscribers on a logged in and logged out Diff.

Diff Detail

Repository

rP Phabricator

Branch

T7317

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4537
Build 4551: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 28448.Feb 18 2015, 7:07 PM

chad added a task: T7317: Can't view other subscribers while logged out.

chad retitled this revision from to Allow public on list of subscribers.

chad updated this object.

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

chad added reviewers: epriestley, btrahan.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptFeb 18 2015, 7:07 PM

epriestley accepted this revision.Feb 18 2015, 7:10 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Feb 18 2015, 7:10 PM

Closed by commit rPf9638edf37a3: Allow public on list of subscribers (authored by chad, committed by chad). · Explain WhyFeb 18 2015, 7:11 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

subscriptions/

controller/

PhabricatorSubscriptionsListController.php

4 lines

Diff 28448

View Options

src/applications/subscriptions/controller/PhabricatorSubscriptionsListController.php

Allow public on list of subscribersClosedPublicActions