review suggestions only shows the first mentor

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: glob, Assigned: glob)

Tracking

Production
x86
macOS
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

the review extension's $bug.mentor field only returns the first mentor it finds.
as some bugs have multiple mentors, it should return an array.
Blocks: 869989
Posted patch 942029_1.patch (obsolete) — Splinter Review
Attachment #8336593 - Flags: review?(dkl)
Comment on attachment 8336593 [details] [diff] [review]
942029_1.patch

Review of attachment 8336593 [details] [diff] [review]:
-----------------------------------------------------------------

Overall works fine and looks good. One issue I found was that if a mentor is included that is also a component/product suggested reviewer, then their name is displayed twice. Please filter out the duplicates with the mentor entry taking precedence.
Attachment #8336593 - Flags: review?(dkl) → review-
Attachment #8336593 - Attachment is obsolete: true
Attachment #8343253 - Flags: review?(dkl)
Comment on attachment 8343253 [details] [diff] [review]
942029_2.patch

Review of attachment 8343253 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl
Attachment #8343253 - Flags: review?(dkl) → review+
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.2/
modified extensions/Review/Extension.pm
modified extensions/Review/lib/WebService.pm
modified extensions/Review/template/en/default/hook/global/header-start.html.tmpl
modified extensions/Review/web/js/review.js
Committed revision 9169.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.