Andreas Wagner

Results 129 comments of Andreas Wagner

From the testing instructions: > choose "set needs human review" action, choose a version, and submit with a comment > navigate to the admin for that add-on (link in the...

Why would we want to disable auto-approval if the submission has already been auto-approved? Do we assume that a potential second version would be significantly more malicious than a first...

@eviljeff please back out https://github.com/mozilla/addons-server/pull/22443 (https://github.com/mozilla/addons-server/commit/15e4bd0b8ee64da8384a5f15b281aa1af734e12c) while the requirements are re-evaluated and redefined. Thank you.

As of today, the following reasons are missing and should be added: * UNKNOWN (can probably be folded into "Other") * SCANNER_ACTION * ABUSE_REPORTS_THRESHOLD * HOTNESS_THRESHOLD * MANUALLY_SET_BY_REVIEWER * INHERITANCE...

> The following are also missing but I am not sure what to do with them: > PENDING_REJECTION_SOURCES_PROVIDED (quite specific) > AUTO_APPROVED_PAST_APPROVAL_DELAY (very specific, are we even still using this...

I'm sorry I have don't follow what this issue is about.

> Moving this to the Reviewer Tools epic as it is not specifically DSA related. Could you expand on that? Resolving appeals looks quite DSA related.

> then the email says the previous decision was correct. Do you mean "incorrect"? The screenshot included in the report indicates the opposite of what you wrote, unless I am...