Closed
Bug 436407
Opened 16 years ago
Closed 13 years ago
Allow editors to mark an add-on in the review queue as "taken"
Categories
(addons.mozilla.org Graveyard :: Admin/Editor Tools, defect, P3)
addons.mozilla.org Graveyard
Admin/Editor Tools
Tracking
(Not tracked)
RESOLVED
FIXED
6.0.8
People
(Reporter: baz, Assigned: gkoberger)
Details
(Whiteboard: [ReviewTeam])
When editors are reviewing add-ons, there may be some "back and forth" conversations with the author. Since there is no explicit "I'm dealing with this add-on" or "taken" flag set on an add-on review, editors' work might get mid-air collision'ed (in Bugzilla terminology). We need a "taken" or "reserved" flag so that editors can indicate to other editors that a particular add-on is being worked on.
Updated•15 years ago
|
Severity: normal → enhancement
Priority: -- → P5
Target Milestone: --- → Future
Comment 2•14 years ago
|
||
This has been brought up multiple times. We should try to implement it soon.
Assignee: nobody → jorge
Severity: enhancement → normal
Status: NEW → ASSIGNED
Priority: P5 → P3
Whiteboard: [required amo-editors]
Target Milestone: Future → 5.12
Version: 3.2 → unspecified
Comment 3•14 years ago
|
||
I see two possible implementations here: 1) "Watch" When an editor is looking at a review page, an AJAX request is sent at regular intervals, telling AMO that the editor is still watching it. We show in the review list and the review page that "X is looking at this review". Multiple editors would be allowed to watch the same review, but they would also be discouraged to perform a review on an add-on somebody else is watching. 2) "Reserve" We add a link or button to review pages where editors can reserve a review for an hour or so. Nobody can perform a review on an add-on reserved by somebody else, and only an admin can override the reservation (the link would turn into something else like "Unreserve" for admins). I think #1 is more practical, and maybe easier to implement.
Assignee: jorge → nobody
Status: ASSIGNED → NEW
Comment 5•14 years ago
|
||
"Multiple editors would be allowed to watch the same review, but they would also be discouraged to perform a review on an add-on somebody else is watching." We'll need to know who is watching first, otherwise everyone will think they are locked out.
Updated•13 years ago
|
Target Milestone: 5.14 → ---
Updated•13 years ago
|
Assignee: nobody → gkoberger
Target Milestone: --- → Q2 2011
Comment 6•13 years ago
|
||
We're aiming for option 1 still, although "watching" sounds long term - let's just say they are looking at the page.
Updated•13 years ago
|
Target Milestone: Q2 2011 → 6.0.8
Assignee | ||
Comment 7•13 years ago
|
||
https://github.com/jbalogh/zamboni/commit/97da9 This will probably need some tweaks once we see how it's used in the wild. A few random thoughts: - We might want a "The status of this review has changed since you loaded this page" indicator; I think that's more important than who's viewing the page. (If you want this, file a bug.) - I only show the person who currently has the "lock" on the page, since showing everyone would have really complicated the code for what would be (in my opinion) an edge case that doesn't add much benefit. If having a list of everyone on the page is vital, feel free to file a bug. - We can play with the wording and styling. You can file a bug, or just comment here and I can change it. - I had a patch that showed an icon on the review log page, but it was freezing the page up so I didn't land it yet. I'm going to optimize that, and it'll land today or I'll file a bug and land it next week.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 8•12 years ago
|
||
Reclassifying editor bugs and changing to a new whiteboard flag. Spam, spam, spam, spam...
Whiteboard: [required amo-editors] → [ReviewTeam]
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•