Closed Bug 1109701 Opened 10 years ago Closed 10 years ago

Admin auto-flag should leave comment on add-on review history

Categories

(addons.mozilla.org Graveyard :: Admin/Editor Tools, defect)

Product:
addons.mozilla.org Graveyard
Component:
Admin/Editor Tools
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
2015-01

People

(Reporter: jorgev, Assigned: magopian)

Details

(Whiteboard: [ReviewTeam:P1])

For the sake of clarity, when we auto admin-flag an add-on, there should be a comment in the review history indicating that it was auto-flagged, and the reason behind it. We've already had several cases where developers included sources when they didn't need to and their add-ons were flagged unnecessarily. Having stated reasons makes it easier to determine if the add-on should be unflagged.
This should use the same approximate methods as the code which adds a comments to auto-admin-flag for admin validation bypass.
Whiteboard: [ReviewTeam] → [ReviewTeam:P1]
Target Milestone: --- → 2014-12
PR https://github.com/mozilla/olympia/pull/404
Assignee: nobody → mathieu
Fixed in https://github.com/mozilla/olympia/commit/6a6fed3a57f97fd74e90c728f073ff50f3b05b44
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: 2014-12 → 2015-01
Verified as fixed : screencast.com/t/SSg4ZArrsEdd
Status: RESOLVED → VERIFIED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.