541221 - Promote more information about the review queues on AMO

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, defect, P3)

Description

[Jorge Villalobos [:jorgev] (he/him)]

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.

Comment 1

[Jorge Villalobos [:jorgev] (he/him)]

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/

Comment 2

[Jorge Villalobos [:jorgev] (he/him)]

Attachment: 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.

Comment 3

[Wil Clouser [:clouserw]]

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.

Comment 4

[Jorge Villalobos [:jorgev] (he/him)]

Attachment: 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.

Comment 5

[Wil Clouser [:clouserw]]

Comment on attachment 432954 [details] [diff] [review]
Patch V2. Moved string changes to the right place and fixed URL handling.

rockin' the sprintf!

Comment 6

[Jorge Villalobos [:jorgev] (he/him)]

Fixed, r64266

Comment 7

[Jorge Villalobos [:jorgev] (he/him)]

Reclassifying editor bugs and changing to a new whiteboard flag. Spam, spam, spam, spam...