Page MenuHomePhabricator

Policy - clean up access to user profile image uri
AbandonedPublic

Authored by btrahan on Feb 2 2015, 9:58 PM.
Tags
None
Referenced Files
F14007113: D11631.diff
Mon, Oct 28, 11:55 PM
F14002008: D11631.id.diff
Fri, Oct 25, 2:13 PM
F13992871: D11631.id.diff
Tue, Oct 22, 7:25 PM
Unknown Object (File)
Oct 10 2024, 10:57 PM
Unknown Object (File)
Sep 26 2024, 10:05 PM
Unknown Object (File)
Sep 21 2024, 11:02 PM
Unknown Object (File)
Aug 27 2024, 5:26 AM
Unknown Object (File)
Aug 25 2024, 7:40 AM
Subscribers

Details

Reviewers
epriestley
Maniphest Tasks
T7094: Clean up T603
Summary

Ref T7094. We already had and were mostly using "needProfileImage" on the people query class. Only real trick in this diff is deleting a conduit end point that has been marked deprecated for the better part of 3 years.

Also includes a small bonus lint fix by adding the $for_app parameter to a child class so as to be consistent with the parent.

Test Plan

clicked around the people action and profiles and calendars loaded nicely.

Diff Detail

Repository
rP Phabricator
Branch
T7094_8
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4256
Build 4269: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan retitled this revision from to Policy - clean up access to user profile image uri.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.