112
edits
Add-ons/Reviewers/Guide/Team: Difference between revisions
→Communication Channels: IRC is dead. Long live Matrix.
Caitmuenster (talk | contribs) (→Enforcement: updated some language) |
Bjherbison (talk | contribs) (→Communication Channels: IRC is dead. Long live Matrix.) |
||
| (5 intermediate revisions by 3 users not shown) | |||
| Line 4: | Line 4: | ||
All reviewers are subscribed to an internal mailing list. On this list we discuss add-on reviews and review policies. If you have questions about a review, you can ask them there. Queue status and reviewer contribution reports are sent weekly. Only members of the list can send messages to it, so it has fairly low activity. | All reviewers are subscribed to an internal mailing list. On this list we discuss add-on reviews and review policies. If you have questions about a review, you can ask them there. Queue status and reviewer contribution reports are sent weekly. Only members of the list can send messages to it, so it has fairly low activity. | ||
For live communication we use [[ | For live communication we use [[Matrix|the #addon-reviewers Matrix channel]]. | ||
Finally, we encourage you to join the [https://mozillians.org/ Mozillians directory]. If you're a reviewer, you can ask on the mailing list or | Finally, we encourage you to join the [https://mozillians.org/ Mozillians directory]. If you're a reviewer, you can ask on the mailing list or Matrix for someone to invite you and vouch for you. Adding the 'addon-reviewers' tag to your account should make you visible on our [https://mozillians.org/en-US/group/addon-reviewers/ Mozillians group]. | ||
= Behavior = | = Behavior = | ||
| Line 23: | Line 23: | ||
==Enforcement== | ==Enforcement== | ||
Volunteer reviewers represent Mozilla in their decisions and interactions with add-on developers, so establishing and enforcing review standards is essential to a good developer experience and user safety. | |||
Here are the types of mistakes we track and their corresponding severity: | Admin reviewers perform regular spot checks of reviews. During these checks, they might find mistakes in review decisions. Here are the types of mistakes we track and their corresponding severity: | ||
'''Most severe''' | '''Most severe''' | ||
| Line 43: | Line 43: | ||
*Incoherent English | *Incoherent English | ||
*Unresponsive or not helpful when involved in conversation with developer | *Unresponsive or not helpful when involved in conversation with developer | ||
If multiple severe and/or frequent mistakes are found, reviewers will receive a notification that they have been put on probation. In probation, an admin will spot-check a reviewer’s activity for six months and remove them from the group if they make one more severe mistake. If the reviewer improves, they will remain in the group conditionally, provided they do not revert to making errors. | |||
Removal of reviewing rights is rare and done only when we feel users may experience a security risk, developers are having a poor experience, or the contributions are causing a drain on resources. If you have any questions or concerns, please contact us in one of the ways listed above. | Removal of reviewing rights is rare and done only when we feel users may experience a security risk, developers are having a poor experience, or the contributions are causing a drain on resources. If you have any questions or concerns, please contact us in one of the ways listed above. | ||
| Line 48: | Line 50: | ||
Thanks for joining the team and supporting the add-on community, our developers and users appreciate your contributions! | Thanks for joining the team and supporting the add-on community, our developers and users appreciate your contributions! | ||
[[Add-ons/Reviewers/Guide/CompleteThemes|Next: Complete Themes]] | [[Add-ons/Reviewers/Guide/Moderation|Previous: Moderation]] [[Add-ons/Reviewers/Guide/CompleteThemes|Next: Complete Themes]] | ||