timomeinen / team-piazza

A build monitor for the TeamCity continuous integration server
GNU General Public License v3.0
17 stars 7 forks source link

Add support for gravatar #40

Closed timomeinen closed 10 years ago

timomeinen commented 10 years ago

From gro...@gmail.com on March 13, 2012 21:53:51

Now that users can (have to) configure their image URLs, I think it makes sense that an admin can configure a server-wide default to use gravatar. This uses the md5 of the user's email address.

Ex: http://www.gravatar.com/avatar/f04729a75b927c0d0657b92857c0482a?s=50&d=identicon I used to just set this in main-config for everyone.

Original issue: http://code.google.com/p/team-piazza/issues/detail?id=41

timomeinen commented 10 years ago

From gro...@gmail.com on March 13, 2012 14:00:56

Workaround to mass update all users with a gravatar URL:

INSERT INTO user_property SELECT id, 'plugin:notificator:piazza:userImage', ' http://www.gravatar.com/avatar/' + substring(master.sys.fn_varbintohexstr(HASHBYTES('md5',lower(email))),3,32) + '?s=50&d=identicon' FROM [users] LEFT JOIN user_property ON users.ID = user_property.user_id AND user_property.key1 = 'plugin:notificator:piazza:userImage' WHERE user_property.value IS NULL

I still consider this issue valid. There are way too many dev tools already that have their own internal profiles/user images, and this just contributes to the problem.

timomeinen commented 10 years ago

From timomeinen on April 13, 2012 01:35:23

You are right. We should add support for gravatar.

In the meantime, you could put the URL to the gravatar image into the notifications configuration.

Status: Accepted
Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-High Usability

timomeinen commented 10 years ago

From timomeinen on October 20, 2013 09:05:09

Status: Started

timomeinen commented 10 years ago

From timomeinen on October 20, 2013 09:17:25

The new release piazza-20132010181251 supports Gravatar. The logic is as follows:

Status: Fixed