755622 - Names in reports page seem to be non searchable

Status: VERIFIED FIXED

(Mozilla Reps Graveyard :: reps.mozilla.org, task)

Description

[Pierros Papadeas [:pierros]]

I tried queries with names of reps filled reports in /reports/ page and the queries returned zero reports (although reports where filled by those names)

Names (full, usernames, local names) of reps and their mentees should count as query items.

Comment 1

[Giorgos Logiotatidis [:giorgos]]

That was intentional and I provided the /mentor/<name> and /rep/<name> pages as an alternative. I believe that adding all these additional search options will result in slower response and more importantly results with lots of noise.

I'll go ahead and implement this as you requested and we can revisit in the future.

Comment 2

[[github robot]]

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

https://github.com/mozilla/remo/commit/417148f7837be44049e093fd02ca1b84988a7ced
Add first_name, last_name, local_name, display_name as search options. Fix bug 755622.

https://github.com/mozilla/remo/commit/95d9ed917ff511a0d7966ab247718384b54745e7
Merge pull request #173 from glogiotatidis/755622-names-in-reports

Add first_name, last_name, local_name, display_name as search options. Fix bug 755622.

Comment 3

[Rebecca Billings [:rbillings]]

Please add details on how to view & search the reports pages- it is unclear how to get to them in order to search.

Comment 4

[Rebecca Billings [:rbillings]]

I'm guessing this isn't referring to mentee individual report pages that you can find via the dashboard if you are using queries.

Comment 5

[Giorgos Logiotatidis [:giorgos]]

This is related to bug 745459.

A user should be able to search based on 'first name', 'last name', 'username' and 'local name' in /reports/ pages.

Thanks!

Comment 6

[Giorgos Logiotatidis [:giorgos]]

I think that I wasn't clear, so here is an example

On

http://reps.allizom.org/reports/

search for 'Rebeca' and you should get reports filed by or mentored by 'Rebeca'.

Comment 7

[Rebecca Billings [:rbillings]]

Verified able to search on the variables listed.