Skip to content

Conversation

@eviljeff
Copy link
Member

@eviljeff eviljeff commented Jan 23, 2026

Fixes: mozilla/addons#15984

Description

Expands the Approve action to do as the naming implies, to approve the listing content (for add-ons). If the add-on is STATUS_REJECTED, approving the content will restore the addon status too (and send an email to the developer informing them).

Context

Even though for content review there won't usually be any abuse reports resolved, the code (and tests) are written to support resolving an abuse report at the same time (which might be useful for mozilla/addons#15969)

Testing

Similar to #24308, this is mainly expected to be used with content review in Cinder, but that doesn't exist yet - though the Approve policy does exist already - so easiest way is to use the abuse report flow:

For content review where the add-on isn't rejected (normal case):

  • identify an add-on that hasn't already been content reviewed (or delete its AddonApprovalsCounter instance)
  • report add-on for abuse
  • in Cinder make a decision to Approve the add-on
  • replay the webhook payload with ./manage.py fake_cinder_webhook
    • the add-on should now be marked as content approved (not in the reviewer tools content review queue)
    • no email should have gone out to the developer

Then for content review where the add-on is rejected (i.e. the developer requested another review):

  • either:
    • go through the reject listing steps (report for abuse; edit/identify a policy so it has amo-reject-listing-content as the action; replay webhook),
    • or just update it's AddonApprovalsCounter so last_content_review_pass = False and the status is STATUS_REJECTED
  • report the add-on for abuse
  • in Cinder make a decision to Approve the add-on again
  • replay the webhook payload with ./manage.py fake_cinder_webhook
    • the add-on should now be marked as content approved, and the status restored to APPROVED (or whatever the previous status was before)
    • an email should have gone out to the developer informing them we restored their listing

Checklist

  • Add #ISSUENUM at the top of your PR to an existing open issue in the mozilla/addons repository.
  • Successfully verified the change locally.
  • The change is covered by automated tests, or otherwise indicated why doing so is unnecessary/impossible.
  • Add before and after screenshots (Only for changes that impact the UI).
  • Add or update relevant docs reflecting the changes made.

@eviljeff eviljeff marked this pull request as ready for review January 23, 2026 13:16
@eviljeff eviljeff requested a review from diox January 23, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task]: implement listing-approve action that is available for cinder policies

1 participant