mozilla / addons

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

[Task]: Ability to Modify a Pending Rejection Date #14920

Open chrstinalin opened 3 months ago

chrstinalin commented 3 months ago

Description

Currently unable to change the date of a pending rejection. If the date needs to be changed, the pending rejection must be deleted and a new pending rejection created in its place. Results in a new rejection email to the developer.

Should have the ability use a date picker to modify the pending rejection date, and either:

  1. Without any additional email, or
  2. With a new email that explains the extended time frame.

Acceptance Criteria

  ### Milestones/checkpoints
  - [ ] The ability to change the pending rejection date, possibly as an inline action
  - [ ] Email handling

Checks

┆Issue is synchronized with this Jira Task

chrstinalin commented 3 months ago

Should the email be optional? Should it be implemented as a inline action (similar to setting due_date or clear pending rejections)?

wagnerand commented 3 months ago

I think given the fact that different versions can have different due dates at the same time, implementing this might not be as straight forward as we initially thought. To do it correctly, you would somehow have to select the versions first for which you want to change the pending rejection date, similar to what you have to do right now when you want to clear the due date.

In any case, any changes affecting the availability or future ability or an add-on or a version, which changing the pending rejection date definitely is, needs to be communicated to the developer. The main goal here was to reduce the effort for reviewers and also send only one email instead of multiple.

However, given the complications mentioned, I am not sure this is something that can be fixed right now, or at least I am not sure how. We can change underlying logic around whether there can be multiple pending rejections (=different dates), but that seems a larger change that needs more consideration.

Thoughts, @diox @mixedpuppy?

diox commented 3 months ago

I agree. Maybe we can address just the simplest scenario though, where all pending rejection dates are the same? It would probably be a specific action with the version dropdown then, and a check when the action is triggered that would throw an error if the current pending rejection dates on the selected versions differ.

diox commented 3 months ago

@wagnerand assigning to you to confirm whether or not my suggestion would solve most cases/be useful enough.

wagnerand commented 3 months ago

@diox I think that would be sufficient enough for most cases. Let's do that.

diox commented 2 months ago

@wagnerand can you provide the email template to use ?

wagnerand commented 2 months ago

I asked Ashley via email for the copy.

wagnerand commented 1 month ago

I asked Ashley to provide the copy in this issue.

aroybalreid commented 1 month ago

Draft copy is below. Please let me know if you see any issues, or if there are any circumstances where we planned to use this correspondence for which this appears inadequate.

Hello,

As you are aware, your extension {{ Add-on name }} was manually reviewed by the Mozilla Add-ons team, at which point we found a violation of one or more Mozilla add-on policies.

Our previous correspondence indicated that you would be required to correct the violation(s) by {{ old deadline }}. However, after further assessing the circumstances - including the violation itself, the risks it presents, and the steps required to resolve it - we have determined that an alternative timeline is appropriate. Based on that determination, we have updated the deadline, and will now require you to correct your add-on violations no later than {{ new deadline }}.

More information about Mozilla’s add-on policies can be found at https://extensionworkshop.com/documentation/publish/add-on-policies/.

Thank you for your attention.

[{{ cinder_job_id }}]

Mozilla Add-ons Team https://a.m.o/