Suggested reviewers should exclude the current user from the list displayed

RESOLVED FIXED

Status

()

bugzilla.mozilla.org
Extensions: Review
--
minor
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: emorley, Assigned: dkl)

Tracking

Production

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
1) Logged in with my account
2) https://bugzilla.mozilla.org/attachment.cgi?bugid=1192957&action=enter
3) Set r?, and open the suggested reviews list

Expected:
Not see my own account suggested as a reviewer

Actual:
My account included in the list.


(Not urgent in the slightest :-))
(Assignee)

Comment 1

2 years ago
Removed from Tree Management :: Treeherder: Docs & Development. I see you are also on several other components in that product as well? Are the others OK?

For a full list: https://bugzilla.mozilla.org/describecomponents.cgi?product=Tree%20Management
Assignee: nobody → dkl
Status: NEW → ASSIGNED
Flags: needinfo?(emorley)
(Reporter)

Comment 2

2 years ago
Ah I actually meant that in general the suggested reviewers feature should only suggest other people, not myself - but I see how that could have been misinterpreted :-)
Flags: needinfo?(emorley)
Summary: Suggested reviewers should exclude myself from the list → Suggested reviewers should exclude the current user from the list displayed
(Assignee)

Comment 3

2 years ago
(In reply to Ed Morley [:emorley] from comment #2)
> Ah I actually meant that in general the suggested reviewers feature should
> only suggest other people, not myself - but I see how that could have been
> misinterpreted :-)

Ah :) Sorry for the confusion. Yeah that does make sense and should be simple enough to fix with a small code change.

dkl
(Assignee)

Comment 4

2 years ago
Created attachment 8667952 [details] [diff] [review]
1209971_1.patch
Attachment #8667952 - Flags: review?(glob)
Comment on attachment 8667952 [details] [diff] [review]
1209971_1.patch

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

r=glob
Attachment #8667952 - Flags: review?(glob) → review+
(Assignee)

Comment 6

2 years ago
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   f992c47..0a62094  master -> master
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
(Reporter)

Comment 7

2 years ago
Many thanks :-)
You need to log in before you can comment on or make changes to this bug.