Closed
Bug 1239838
Opened 9 years ago
Closed 9 years ago
Don't see a way to redirect a needinfo request (in Experimental UI)
Categories
(bugzilla.mozilla.org :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bzbarsky, Unassigned)
References
(Blocks 2 open bugs)
Details
Attachments
(2 files)
18.08 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
34.56 KB,
image/png
|
Details |
In the non-experimental UI, needinfo requests are surfaced in two bits of UI:
1) An "I am providing ..." checkbox under the comment area and a "Need more information" checkbox under the comment area, with the latter allowing you to select whom to ask for info.
2) A flag in the "Flags" section that can be set to "?" and then you enter whom you want info from.
In the experimental UI, UI #2 seems to be gone. There's an "Update" button in the "People" section which just jumps you to UI #1.
The problem is that UI #1 doesn't allow _redirecting_ a needinfo request: changing the requestee while leaving the requester the same. The idea is to have the original requester notified when the request is responded to, instead of whoever is making the change getting notified. With the non-experimental UI, this can be done in a slightly hacky way as follows:
* Change the username listed in the Flags section
* Ensure the "I am providing..." checkbox (which is checked by default if
you're the requestee) is unchecked.
In the experimental UI I don't see a way to do this at all. This is a pretty common operation on needinfo requests that I think we should add better support for in general, not remove support for.
I just ran into this as well, and needed to turn off experimental UI in order to make bug 1250824 comment 3, and then turn it back on.
Summary: Don't see a way to redirect a needinfo request → Don't see a way to redirect a needinfo request (in Experimental UI)
unchecking your "clear needinfo request" changes 'Need more information from' to 'Redirect my needinfo request to'
- YUI --> jquery
- move all js into a single script block
- implement redirection
Attachment #8741646 -
Flags: review?(dkl)
Comment 4•9 years ago
|
||
Comment on attachment 8741646 [details] [diff] [review]
1239838_1.patch
Review of attachment 8741646 [details] [diff] [review]:
-----------------------------------------------------------------
r=dkl
I tested existing flags that are requesteeable with changing the requestee to multiple addresses and it does indeed work. The first address will take over the same flag id and the following addresses will get new flags. This works for both needinfo, review, etc. We could document this behavior or we could leave it alone in case we want to remove this behavior in the future for whatever reason.
dkl
Attachment #8741646 -
Flags: review?(dkl) → review+
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
8d6c922..d447aed master -> master
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Depends on: 1269939
Depends on: 1280989
Updated•5 years ago
|
Component: User Interface: Modal → User Interface
Updated•5 years ago
|
Assignee: glob → nobody
You need to log in
before you can comment on or make changes to this bug.
Description
•