Versions mockup missing several fields

RESOLVED FIXED in 5.12

Status

defect
P1
normal
RESOLVED FIXED
9 years ago
3 years ago

People

(Reporter: clouserw, Assigned: chowse)

Tracking

unspecified
5.12

Details

Reporter

Description

9 years ago
Example of our current page: https://addons.mozilla.org/en-US/developers/versions/edit/51899

The new mockup: http://people.mozilla.com/~chowse/drop/amo/devtools/v2/Addon_05_Versions_Edit.png

The current page has:
- ability to add localized release notes
- ability to edit the license for that version
- an "approval notes" field

-> chowse for mockup alterations
Assignee

Comment 1

9 years ago
(In reply to comment #0)
> The current page has:
> - ability to add localized release notes

The Release Notes field will be localizable in the same manner as other DevTools pages: by using the locale field in the top-right corner of the page:

http://people.mozilla.com/~chowse/drop/amo/devtools/v2/Addon_01_Common.png


> - ability to edit the license for that version

Discussed in bug 580031. Per-version license editing is going away; you can only change the license of the current version, using the "Manage Authors & License" page:

http://people.mozilla.com/~chowse/drop/amo/devtools/v2/Addon_03_Authors.png

If we preserve the license on this page, it should be read-only. There could be some UI here for admin changes, but I'm not sure if there's a frequent-enough need for it.


> - an "approval notes" field

I will add this.
Assignee

Comment 2

9 years ago
Update mock:
http://people.mozilla.com/~chowse/drop/amo/devtools/v2/Addon_05_Versions_Edit.png

Two questions:

* Any insights into how much or what kind of content the 'Approval Information'
  field can contain? I wasn't entirely clear, so I just copied it verbatim for
  the moment.

* Does 'Approver Notes' need to be a part of the New Version workflow?
Reporter

Comment 3

9 years ago
I think the read-only license looks good.

Approval notes is for if a reviewer needs to have something installed to make the add-on work, or if they need a user/pass for a site, etc.  It's unlimited length.  It should be a part of the "new version" workflow.
Assignee

Comment 4

9 years ago
How about the 'Approval Information' part: the one with file submission and approval dates? Will it list that info for each file? If a file/version hasn't been approved, what will it display?
Reporter

Comment 5

9 years ago
that info is displayed regardless of approval status.  It should also be editable at any time for any version.
Assignee

Comment 6

9 years ago
Editable? I don't think we're talking about the same thing here. I'm not referring to the 'Approver Notes', but the 'Approval Information' seen here in the current DevTools:

http://img.skitch.com/20100922-edcukbp336mc88tastc2t7dgy2.png

And here in the new mocks:

http://img.skitch.com/20100922-tht5fk12hgyyqn5arnww6wttkw.png

I think it might make more sense to have that information up in file table at the top of the page, but I'm just not sure what information can show up there (besides what's shown in the version you linked to).
Reporter

Comment 7

9 years ago
Oh.  That is not...pretty.  The file id?  really? heh

I think the only thing that could change there is the (ALL) - which could change to the platform specific options like Windows, or "Public" which could be sandbox, etc.
Assignee

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Reporter

Comment 8

9 years ago
Alright, we've had the file id this long, won't hurt to keep it around a little more.  If someone gets a bright idea for what to do with that row, let's chat, otherwise -> Fixed.  Thanks.
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.