Promote more information about the review queues on AMO

RESOLVED FIXED in 5.8

Status

addons.mozilla.org Graveyard
Developer Pages
P3
normal
RESOLVED FIXED
8 years ago
2 years ago

People

(Reporter: jorgev, Assigned: jorgev)

Tracking

unspecified

Details

(Whiteboard: [ReviewTeam])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

8 years ago
There are a couple of important resources that authors should be made aware of:
* The information for authors in the editor wiki. https://wiki.mozilla.org/AMO:Editors/InfoAuthors
* The weekly queue reports in the AMO Forum. https://forums.addons.mozilla.org/viewforum.php?f=21

These should probably appear in the last step of the submission process, and maybe in the status page when the add-on is pending a nomination review.
(Assignee)

Updated

8 years ago
Summary: Promote more information about reviews on AMO → Promote more information about the review queues on AMO
(Assignee)

Comment 1

8 years ago
The last step of the submission is bug 541591. I'll solve the status page on this bug. A better resource to point all developers is this recent post: http://blog.mozilla.com/addons/2010/02/15/the-add-on-review-process-and-you/
(Assignee)

Comment 2

8 years ago
Created attachment 427019 [details] [diff] [review]
Patch V1. Add link to blog in status page and minor correction

This adds a link to the blog post explaining the review process. I've indicated the post is in English in order to be a little more locale-friendly.

I also made a minor correction to the string about the sandbox, which said users needed to log in in order to install the add-on, which is no longer true.
Attachment #427019 - Flags: review?(clouserw)
Attachment #427019 - Flags: review?(clouserw) → review-
Comment on attachment 427019 [details] [diff] [review]
Patch V1. Add link to blog in status page and minor correction

The .pot file is automatically generated.  You'll need to change these strings in their respective files.  You can see the file name right above the string.

In this case, we should add lang="en" to the <a> and then you won't need "in English" afterwards - localizers can add that if they really think it's useful.  Secondly, instead of the whole URL in the .po, if you use a %s we can substitute it in from the code side and keep the .po files simpler.  If you don't know how to do that assign this to me and I can do it.
Target Milestone: 5.7 → 5.8
(Assignee)

Comment 4

8 years ago
Created attachment 432954 [details] [diff] [review]
Patch V2. Moved string changes to the right place and fixed URL handling.

Moved the string changes to the right files and separated the URL from the strings. Also added lang="en" to the links.
Attachment #427019 - Attachment is obsolete: true
Attachment #432954 - Flags: review?(clouserw)
Comment on attachment 432954 [details] [diff] [review]
Patch V2. Moved string changes to the right place and fixed URL handling.

rockin' the sprintf!
Attachment #432954 - Flags: review?(clouserw) → review+
(Assignee)

Comment 6

8 years ago
Fixed, r64266
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

6 years ago
Reclassifying editor bugs and changing to a new whiteboard flag. Spam, spam, spam, spam...
Whiteboard: [required amo-editors] → [ReviewTeam]
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.