Remove gravatar support
ClosedPublic
Actions

Authored by btrahan on Oct 17 2014, 4:47 PM.

Details

Reviewers

epriestley
chad

Maniphest Tasks

T6261: Remove Gravatar support to reduce setup complexity on hosts without access to the internet

Commits

Restricted Diffusion Commit
rP9dd09f717192: Remove gravatar support

Summary

Fixes T6261. The performance of asking gravatar for these images is horrible and causing lots of people to have issues with the page.

Test Plan

noted how wildly fast the edit profile picture page loaded

Diff Detail

Repository

rP Phabricator

Branch

T6261

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2862
Build 2866: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 25748.Oct 17 2014, 4:47 PM

btrahan added a task: T6261: Remove Gravatar support to reduce setup complexity on hosts without access to the internet.

btrahan retitled this revision from to Remove gravatar support.

btrahan updated this object.

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

btrahan added reviewers: epriestley, chad.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptOct 17 2014, 4:47 PM

Harbormaster completed remote builds in B2862: Diff 25748.Oct 17 2014, 4:48 PM

epriestley accepted this revision.Oct 17 2014, 4:49 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Oct 17 2014, 4:49 PM

Closed by commit rP9dd09f717192: Remove gravatar support (authored by btrahan, committed by btrahan). · Explain WhyOct 17 2014, 4:50 PM

This revision was automatically updated to reflect the committed changes.

avivey awarded a token.Oct 17 2014, 5:02 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

people/

controller/

PhabricatorPeopleProfilePictureController.php

45 lines

Diff 25748

View Options

src/applications/people/controller/PhabricatorPeopleProfilePictureController.php

Remove gravatar supportClosedPublicActions