Add-ons/Reviewers/Guide/Team: Difference between revisions

← Older edit

Add-ons/Reviewers/Guide/Team (view source)

Revision as of 12:14, 6 March 2020

483 bytes removed , 6 March 2020

→‎Communication Channels: IRC is dead. Long live Matrix.

Bjherbison

113

edits

@@ Line 2: / Line 2: @@
 = Communication Channels =
-There are 2 mailing lists related to our group:
+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.
-*'''Internal:''' 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 [[Matrix|the #addon-reviewers Matrix channel]].
-*'''Public:''' The other list, amo-editors AT mozilla DOT org, is public. Review emails sent to developers have this list in the Reply-To header, so any objections or comments about add-on reviews are sent there. It is normally a very active list, getting dozens of messages daily. Since most of the messages aren't really relevant to most reviewers, this list is opt-in. If you want to, just request to be added and you will be. Every now and then a review reply sent to this list will require your intervention, and in those cases an admin will add your email address to the conversation thread. If you don't want your email address to be shared to other developers, please indicate this to your guide.
-For live communication we use [[IRC|the #addon-reviewers IRC 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 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].
-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 IRC for someone to invite you and vouch for you. Adding the amo-editors tag to your account should make you visible on our [https://mozillians.org/en-US/group/amo-editors/ Mozillians group].
 = Behavior =
@@ Line 26: / Line 23: @@
 ==Enforcement==
-Unfortunately, there are times when we have to remove reviewers from this group because of egregious and/or frequent violations of reviewing guidelines. Reviewers will receive a warning if they are close to being put on probation. In probation, an admin will spot-check a reviewer’s activity for a month and remove them from the group if they continue making mistakes. If the reviewer improves, they will remain in the group conditionally for the next six months, provided they do not revert to making errors.
+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'''
@@ Line 47: / Line 44: @@
 *Unresponsive or not helpful when involved in conversation with developer
-Removal of reviewing rights is rare and done only when we feel users may experience a security risk, developers are having a terrible 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.
+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.
 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]]