canmove, Confirmed users
1,448
edits
Add-ons/Reviewers/Guide/Tools: Difference between revisions
Jump to navigation
Jump to search
Proofreading
(Merged First Steps into Tools) |
(Proofreading) |
||
| Line 1: | Line 1: | ||
New reviewers need to be given permission by an admin to gain access to the review tools. Once the admin has given you permission, you should see an Editor Tools item in the Tools menu on the top right of the AMO homepage. ''Editors'' is an old name given to our team, | New reviewers need to be given permission by an admin to gain access to the review tools. Once the admin has given you permission, you should see an Editor Tools item in the Tools menu on the top right of the AMO homepage. ''Editors'' is an old name given to our team, still used around the site and on our mailing lists. We prefer the term ''Reviewers'' now. | ||
[[Image:Editor-tools.png|center|Editor Tools menu item]] | [[Image:Editor-tools.png|center|Editor Tools menu item]] | ||
If | If the Editor Tools option doesn't appear on the dropdown menu, try logging out and logging back in. If it still doesn't appear, contact the admin who should have given you access. Clicking on the menu item will take you to the reviewers homepage. This page displays the state of the review queues along with some stats. | ||
= Basic tools = | = Basic tools = | ||
| Line 66: | Line 66: | ||
The All Platforms link points to the file to review, generally an XPI. The Validation link will run the automatic code validator on that file and show you the results. The Contents link opens the source code viewer, and the Compare link opens the diff viewer, so you can compare the current version with the previous ones. | The All Platforms link points to the file to review, generally an XPI. The Validation link will run the automatic code validator on that file and show you the results. The Contents link opens the source code viewer, and the Compare link opens the diff viewer, so you can compare the current version with the previous ones. | ||
History entries can have Version Notes and Notes to Reviewers. Version Notes are meant to be public, while Notes to Reviewers are meant for | History entries can have Version Notes and Notes to Reviewers. Version Notes are meant to be public, while Notes to Reviewers are meant for reviewers only. They can include testing information or responses to previous review notes. Since testing information isn't usually version-dependent, we prefer to have those in the Whiteboard field, which you can find at the bottom of review pages. This is a shared field between the add-on developer and the review team, so make sure that testing info is always there and up to date. | ||
== Review actions == | == Review actions == | ||
| Line 74: | Line 74: | ||
Clicking on any of them will open a form below it. | Clicking on any of them will open a form below it. | ||
The comments textbox is where you should write all of your review notes. There's also a canned response list below it, that contains very useful, reusable snippets of text for your notes. Selecting any of them will | The comments textbox is where you should write all of your review notes. There's also a canned response list below it, that contains very useful, reusable snippets of text for your notes. Selecting any of them will add the text to the textbox, wherever the cursor is. You can use as many as you need. | ||
The first 3 actions will resolve the pending review: | The first 3 actions will resolve the pending review: | ||