Drone Wall should be able to display individual avatars for paired commits

[cantidio]

This is more like a question.
Nowadays git cannot handle pairing the right way, instead what is done today is: we change the email before the commit is made to person1+person2@email.com. This way we cannot display a correct image for the pair in the wall.

PS: what people actually do is: hack that into Gravatar and create an association with each pair email. That's not viable while working on a big company and pairing with a lot of different people.

It would be awesome if drone wall could get the author email, split it by "+", and be able to get an image for each person of that particular commit. Of course this could be a option that could be turned on and off. But in theory it seems simple to be implemented.

What do you guys think?

[scottferg]

What's the likelihood of somebody using a + in their email? I honestly
don't know, but if it's common it would probably break for others what it
fixes for some.
On Sep 12, 2014 4:17 PM, "Cantidio Fontes" notifications@github.com
wrote:

This is more like a question.
Nowadays git cannot handle pairing the right way, instead what is done
today is: we change the email before the commit is made to
person1+person2@email.com. This way we cannot display a correct image for
the pair in the wall.

It would be awesome if drone wall could get the author email, split it by
"+", and be able to get an image for each person of that particular commit.
Of course this could be a option that could be turned on and off. But in
theory it seems simple to be implemented.

What do you guys think?

—
Reply to this email directly or view it on GitHub
#17.

[cantidio]

It's not very likely actually. It's used more by people that pair program. There are several tools that are used for changing the email for the commit.

Example: https://github.com/henrik/dotfiles/blob/master/bash/lib/gitpair.sh

People usually use some prefix to their email, like pair+, or others.
If this feature could be configurable it would be awesome. We could have something in the command line like.

./drone-wall --datasource=/var/lib/drone/drone.sqlite --pair_gravatar_prefix="Email_prefix" ...

We could check for this value, if present then break the emails and check the gravatars.
What do you think?

[scottferg]

Sounds reasonable. We're in the process of a total overhaul to the UI, so I'd say let's hold off on this until after next week.