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
F15467066: D11631.id27988.diff
Thu, Apr 3, 12:28 PM
F15466625: D11631.id27988.diff
Thu, Apr 3, 7:44 AM
F15464039: D11631.diff
Wed, Apr 2, 5:44 AM
F15459704: D11631.id27988.diff
Mon, Mar 31, 3:21 PM
F15436463: D11631.id27988.diff
Tue, Mar 25, 2:23 PM
F15392723: D11631.diff
Sat, Mar 15, 5:15 PM
Unknown Object (File)
Feb 21 2025, 4:14 AM
Unknown Object (File)
Feb 8 2025, 2:14 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.