455686 - UI changes to AMO admin tools - Display date and hyperlink to a review

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Admin/Editor Tools, defect)

Description

[Rey Bango]

In looking at the following page, there doesn't seem to be a way to determine when an update to an add-on occurred apart from going into each review record:

https://addons.mozilla.org/en-US/admin/addons?q=[6061]

I would like to suggest the following updates to the add-on manager page:

1) Display the date of the add-on's update next to each review number

e.g.: Version 51893: 0.25 (2008-08-12 13:03:19)

2) Create a hyperlink from the review number to the editor review page. For example, review 50832 would link to

https://addons.mozilla.org/en-US/editors/review/50832

It seems from a high-level that these would be very minor UI changes that would make things easier when investigating issues with add-ons.

Comment 1

[Jeff Balogh (:jbalogh)]

Attachment: hyperlinking the version number and displaying date modified on admin/addons/<id>

Comment 2

[Jeff Balogh (:jbalogh)]

Attachment: Screenshot of proposed UI changes

Rey, let me know what you think about the changes.

Comment 3

[Jeff Balogh (:jbalogh)]

Attachment: Linking the version number and displaying date modified on admin/addons_status page.

Comment 4

[Wil Clouser [:clouserw]]

Comment on attachment 361679 [details] [diff] [review]
Linking the version number and displaying date modified on admin/addons_status page.

r+ if you change
+                          <span class='version-modified'>({$version['modified']})</span></li>";

to something like

+                          <span class='version-modified'>(".strftime(___('datetime'),strtotime($version['modified'])).")</span></li>";

Comment 5

[Rey Bango]

@Jeff: UI change looks great & exactly what I was hoping for. Just to be clear, the date displayed is the date that the update was submitted, correct?

Comment 6

[Jeff Balogh (:jbalogh)]

@Rey: The date getting shown is 'last modified', but I can switch to the date created if that works better for you.

Comment 7

[Rey Bango]

@Jeff: nope that's just fine. :) Thanks

Comment 8

[Jeff Balogh (:jbalogh)]

r22169

Comment 9

[Stephen Donner [:stephend] Not actively reading bugmail]

Looks fine to me on https://preview.addons.mozilla.org/en-US/admin/addons?q=FoxSaver+[5276].

Verified FIXED.

Comment 10

[Rey Bango]

Love it. Great job.