mozilla / addons

☂ Umbrella repository for Mozilla Addons ✨
Other
126 stars 41 forks source link

Temporarily disable auto-approval when instant-rejecting #8301

Closed wagnerand closed 3 years ago

wagnerand commented 3 years ago

Describe the problem and steps to reproduce it:

For a delayed reject, we disable auto-approval until the next manual approval and automatically subscribe the user to submission notifications of the channel for which they rejected versions.

We should do the same for instant-rejects.

STR:

  1. Have a public add-on.
  2. Instant-reject ("Reject immediately") a version.

What happened?

  1. (Note you need to go back to or reload the review page for the add-on to see potential changes.)
  2. "Notify me about new <listed|unlisted> versions" is not checked
  3. "Disable Listed Auto-Approval Until Next Manual Approval" is not changed to "Enable Listed Auto-Approval Until Next Manual Approval"

What did you expect to happen?

  1. (Note you need to go back to or reload the review page for the add-on to see potential changes.)
  2. "Notify me about new <listed|unlisted> versions" got checked automatically
  3. "Disable Listed Auto-Approval Until Next Manual Approval" has changed to "Enable Listed Auto-Approval Until Next Manual Approval"

Anything else we should know?

(Please include a link to the page, screenshots and any relevant files.)

AlexandraMoga commented 3 years ago

I've verified this issue on -dev and I have the following results:

wagnerand commented 3 years ago

I forgot to reply here, I filed mozilla/addons#8353 as a followup. Thank you @AlexandraMoga for calling that out!