1038211 - Filter admin UserProfiles by legacy vouches

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[John Giannelos [:nemo-yiannis]]

We need to have a way to check /admin for users with only legacy vouches.

Comment 1

[[github robot]]

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

https://github.com/mozilla/mozillians/commit/133d2e4833598f4763394df2fe4b427f14dd330d
[Fix bug 1038211] Filter admin UserProfiles by users with only legacy vouches.

https://github.com/mozilla/mozillians/commit/efc299ad57c4f4ad32c636ab7c312340dcadb51e
Merge pull request #1039 from johngian/fix-admin-legacy

[Fix bug 1038211] Filter admin UserProfiles by legacy vouches.

Comment 2

[Giorgos Logiotatidis [:giorgos]]

Minimum verification steps:
 1. Go to /admin/users/userprofile/
 2. Right column with filters should include a "By vouch type" filter
 3. Available filter options "All", "Legacy", "New"
 4. Filtering by "Legacy" should return vouched users with _only_ legacy vouches (i.e. vouches migrated by bug 997425)
 5. Filtering by "New" should return vouched users with _at least_ one new vouch
 6. Legacy + New = All

This filter will help us monitor the users who will get unvouched when we go forward with bug 1033305

Comment 3

[William Reynolds [:williamr]]

The filter does not appear to work on the dev environment. When using the link below, the page takes over a minute to load and then displays the message: "Service Unavailable - The service is temporarily unavailable. Please try again later."

https://mozillians-dev.allizom.org/admin/users/userprofile/?vouch_type=legacy

Comment 4

[John Giannelos [:nemo-yiannis]]

I've tested it locally with a database dump from dev and it seems to be working just fine. After some investigation I think that's happening because dev deployment has limited resources. I suggest we mark that as qa- and see how that goes in prod. Since it's not blocking that release (it's temporary code and it doesn't affect any other site functionality) we can reopen it in the future.

:williamr
What do you think?

Comment 5

[[github robot]]

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

https://github.com/mozilla/mozillians/commit/b57c90dfaddd0dabf4d4b4c704e97fb39afa6845
[bug 1038211] Optimize legacy vouches /admin filter.

https://github.com/mozilla/mozillians/commit/dae454c1b5279bfe336b41bdd92bc996aca3e766
Merge pull request #1051 from glogiotatidis/query

[bug 1038211] Optimize legacy vouches /admin filter.

Comment 6

[Giorgos Logiotatidis [:giorgos]]

We worked with :nemo and fixed that, yay!

Note on "temporary code" of comment #4: 
 - This is code to support the transitional phase of the first 60 days of the new vouching system. After we close bug 1033305 this filter will be deleted.

Comment 7

[William Reynolds [:williamr]]

Verified fixed on stage. Thanks Nemo and Giorgos!