[Approval Queue] UI for the Approval Queue Log

RESOLVED WONTFIX

Status

addons.mozilla.org Graveyard
Admin/Editor Tools
--
enhancement
RESOLVED WONTFIX
14 years ago
2 years ago

People

(Reporter: alanjstr, Assigned: fligtar)

Tracking

Dependency tree / graph

Details

Attachments

(1 attachment)

(Reporter)

Description

14 years ago
The approval log should be browsable, showing who approved/denied and when they
did it.

Comment 1

14 years ago
Agreed. :-) This wasn't done originally cause the approval logging is kinda
hackish to start with.
Summary: Browsable approval log → UI for the Approval Log for Admins

Updated

14 years ago
Summary: UI for the Approval Log for Admins → [Approval Queue] UI for the Approval Queue Log

Comment 2

14 years ago
I've done a basic UI. It's basically a tabular print of the table. Anything more
featured will have to wait till after 1.0.
Whiteboard: basic-UI done (fixed-development), after-1.0 for complete.

Updated

14 years ago
Whiteboard: basic-UI done (fixed-development), after-1.0 for complete. → basic-UI done (fixed-beta), after-1.0 for complete.

Comment 3

14 years ago
A basic UI is in update-beta. I'll consider taking a more complete UI for
update-beta if it gets done, otherwise this is post 1.0. 

Marking assigned as this is part of my fixed-beta list (for the basic UI
anyway). and I don't really see a point in spinning of a new bug for the full thing.
Status: NEW → ASSIGNED
Keywords: helpwanted
(Reporter)

Comment 4

14 years ago
I'd like to see pagination and filtering/searching.  I'd prefer it shows Who,
What, When with the ability to click an item to see actual approval/denial
details.  

Comment 5

14 years ago
Bulk Moving Developer Control Panel bugs to new component.
(Filter: massdevcpspam)
Component: Update → Developers
Product: mozilla.org → Update
Version: other → unspecified
(Reporter)

Comment 6

14 years ago
Mass-resolving bugs that have been fixed on trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED

Comment 7

14 years ago
Sorry for the bugspam, reopening bugs wrongly marked as resolved.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Updated

14 years ago
Status: REOPENED → ASSIGNED

Updated

14 years ago
Target Milestone: --- → 1.0

Updated

14 years ago
Whiteboard: basic-UI done (fixed-beta), after-1.0 for complete. → basic-UI done (beta), after-1.0 for complete.
(Reporter)

Comment 8

14 years ago
The column headers for Install? 	Unistall? 	New Chrome? 	App Works? are useless,
since you have to select YES in order to approve.

Comment 9

14 years ago
They're not there for YES, they're there for No. :-)
(Reporter)

Comment 10

14 years ago
But I can't leave a box unchecked, can I?  So it would never be a No

Comment 11

14 years ago
If you're donig a denial, it can be a no. :-)
(Reporter)

Comment 12

14 years ago
Instead of ID and vID, how about the name and version with a link to edit it?  

There are no checkboxes for Visual Errors? and All Elements Themed?, so they can
be removed.  

Maybe rename the values in Action as "Approved", "Denied" by wrapping the print
statement with a replace().  Or maybe a smiley, a frowny, and a circle with a
question mark in it.

I'm not sure what uID is supposed to be.  It doesn't appear to be unique.

Comment 13

14 years ago
Visual Errors / All Elements Themed, as the names suggest are for Themes only.
:-) They don't appear for Extensions.

--> 1.1.
Target Milestone: 1.0 → 1.1

Updated

14 years ago
Assignee: psychoticwolf → nobody
Status: ASSIGNED → NEW
Whiteboard: basic-UI done (beta), after-1.0 for complete.
(Reporter)

Updated

13 years ago
Target Milestone: 1.1 → 2.0

Comment 14

13 years ago
Now that Firefox and UMO are becoming more popular, the approval log is getting
pretty huge. Ability to view pages of 20 items at a time would be useful.

Comment 15

13 years ago
This definetly needs to be improved... I'll work on it by the end of the week.
Assignee: nobody → colin.ogilvie
(Reporter)

Updated

13 years ago
Assignee: colin.ogilvie → jstritar
Keywords: helpwanted

Comment 16

13 years ago
I've started to do some work on this. Colin, if you have done anything or have
any  ideas we can coordinate in irc.
Status: NEW → ASSIGNED

Comment 17

13 years ago
Created attachment 192052 [details] [diff] [review]
Improved approval log UI

Here is what is in this patch:

1) Simplified approval log table which includes important data (removes the
install?, new chrome?, etc columns because the descriptions cover that) and has
borders for easier reading.
2) The log now resolves the IDs to actual extension and user names.
3) Extension and user names are links to edit the user/extension or just to
view the extension (on the moreinfo.php page) if you're not an admin.
4) I've also changed the link in the approval queue to point to the
moreinfo.php for the extension if you do not have privileges to edit it.

CTho: I tried to add more context to the patch, but when I used the actual cvs
command to diff, it thought I changed the entire file... so instead I just made
it with eclipse and the default amount of context.
Attachment #192052 - Flags: first-review?(cst)
(Reporter)

Comment 18

13 years ago
Comment on attachment 192052 [details] [diff] [review]
Improved approval log UI

>@@ -243,79 +256,136 @@
> 
> <?php 
> } else if ($function=="approvalhistory") {
>+
>+function renderPageNav($page, $pages, $per_page) {
>+	$url = "?function=".$_GET['function']."&amp;numpg=".$per_page."&amp;pageid=";

That looks like a function inside a IF block.  Say it aint' so.
Comment on attachment 192052 [details] [diff] [review]
Improved approval log UI

UMO v2 is live.  I had a lot less to do with its development, so check with morgamic before using me for reviews.
Attachment #192052 - Flags: first-review?(cst)

Updated

12 years ago
Depends on: 335410

Comment 20

12 years ago
AMO bugspam. Correcting QA contacts on OLD bugs (mozilla.update@update.bugs)

-> Correct QA contact (developers@add-ons.bugs)

Filtermeplzkthx
QA Contact: mozilla.update → developers
(Assignee)

Updated

12 years ago
Depends on: 352560
(Assignee)

Updated

12 years ago
Assignee: jstritar → fligtar
Status: ASSIGNED → NEW
Target Milestone: 2.0 → 3.0
(Assignee)

Updated

12 years ago
Component: Developer Pages → Admin/Reviewer Tools
QA Contact: developers → admin-tools
Target Milestone: 3.0 → ---
Version: unspecified → 1.0
v2 didn't have a new queue UI, but we're not going to be fixing the v1 UI either.  WONTFIXing rather than making a strict "WFM" reading of the Summary!
Status: NEW → RESOLVED
Last Resolved: 14 years ago12 years ago
Resolution: --- → WONTFIX
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.