854844 - [AP] Show number of vouchees

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[Giorgos Logiotatidis [:giorgos]]

In Admin Panel User List show number of vouchees per User. Admin should be able to sort by number of vouchees.

Comment 1

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/747d20db95fc6ef3fc3b44ea78fb479578b9ba6c
[fix bug 854844] Show number of vouchees per user in Admin Panel.

https://github.com/mozilla/mozillians/commit/ae28183ae0996ee762acfa48a3ee1f49202a076f
Merge pull request #443 from glogiotatidis/vouchees_column

[fix bug 854844] Show number of vouchees per user in Admin Panel.

Comment 2

[Matt Brandt [:mbrandt]]

Attachment: qa - reopened

Reopening:

A small nit - s/vouchees/vouches/

Comment 3

[Giorgos Logiotatidis [:giorgos]]

OK help me fix my english too :)

1. If I want to say 'The number of OKs I've given to people' -> 'The number of vouches'
2. If I want to say 'The number of people I've given OK to' -> 'The number of vouchees'

This column lists the number of people I've given OK to.

Comment 4

[Andrei [:sancus]]

We all agree that Giorgos' has it right, and this will get changed in the future when we switch to invites instead of vouches anyway so don't worry about it ;)

Comment 5

[Matt Brandt [:mbrandt]]

QA verified on stage - attachment 732914 [details] - thanks sancus et al for engaging in a rather in-depth and informative bit of banter in irc.

'vouchees' is correct!

Comment 6

[Andrei [:sancus]]

I removed this fix because it was generating monster queries per Bug 861956. We can refix it with a more efficient custom query.

Comment 7

[Andrei [:sancus]]

Fix reapplied, and situation is resolved.

Comment 8

[Matt Brandt [:mbrandt]]

QA verified on stage - thanks sancus