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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 28449

View Options

src/applications/subscriptions/controller/PhabricatorSubscriptionsListController.php

Allow public on list of subscribersClosedPublicActions