Allow logged-out access to individual diffs (vs revisions)
ClosedPublic
Actions

Authored by epriestley on Jun 10 2014, 1:00 PM.

Details

Reviewers

btrahan
chad

Maniphest Tasks

T5303: Differential is set to public access (without login), but accessing diffs requires login

Commits

Restricted Diffusion Commit
rPc483877f5791: Allow logged-out access to individual diffs (vs revisions)

Summary

Fixes T5303. Individual diffs can have public access policies.

Test Plan

Viewed a public diff while logged out.

Diff Detail

Repository

rP Phabricator

Branch

publicdiff

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 956
Build 956: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 22559.Jun 10 2014, 1:00 PM

epriestley retitled this revision from to Allow logged-out access to individual diffs (vs revisions).

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T5303: Differential is set to public access (without login), but accessing diffs requires login.

Herald added a subscriber: epriestley. · View Herald TranscriptJun 10 2014, 1:00 PM

Harbormaster completed remote builds in B956: Diff 22559.Jun 10 2014, 1:01 PM

chad accepted this revision.Jun 10 2014, 2:14 PM

chad added a reviewer: chad.

This revision is now accepted and ready to land.Jun 10 2014, 2:14 PM

Closed by commit rPc483877f5791 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

controller/

DifferentialDiffViewController.php

4 lines

Diff 22559

View Options

src/applications/differential/controller/DifferentialDiffViewController.php

Allow logged-out access to individual diffs (vs revisions)ClosedPublicActions