Use the directory avatar service #965

Merged
merged 1 commit into from Feb 26, 2016

Conversation

Projects
None yet
2 participants
@kabel
Contributor

kabel commented Feb 26, 2016

This will bring in profile icons from the avatar services supported
through directory.

Use the directory avatar service
This will bring in profile icons from the avatar services supported
through directory.
@mfairchild365

This comment has been minimized.

Show comment
Hide comment
@mfairchild365

mfairchild365 Feb 26, 2016

Member

I think this is fine. It might be worth noting that this will increase load on directory.unl.edu. It might be worth caching the effective URL in directory's avatar service (but that has complications too).

Member

mfairchild365 commented Feb 26, 2016

I think this is fine. It might be worth noting that this will increase load on directory.unl.edu. It might be worth caching the effective URL in directory's avatar service (but that has complications too).

mfairchild365 added a commit that referenced this pull request Feb 26, 2016

Merge pull request #965 from unl/avatar-service
Use the directory avatar service

@mfairchild365 mfairchild365 merged commit d05d8ab into develop Feb 26, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@kabel kabel deleted the avatar-service branch Mar 1, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment