New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DSA: Move force-disable button into reviewer action #9330
Comments
@wagnerand we should replace the force-enable button too - they're a toggle so makes sense to do the same for the opposite action. I'm assuming we don't want to email developers if we force enable? |
@eviljeff Yes, we should replace the force-enable button as well. I don't believe we need developer communication since that does not do anything to the versions themselves (...unless we start tracking which versions have been disabled during force-disable and then re-enable those with force-enable, this is probably out of scope for this bug though). Can we make the comment mandatory and just handle it like an internal comment (i.e. don't send it to anyone)? I don't like that it's a bit inconsistent because I suppose we are sending the force-disable comment to the developer, but it's still better than having no documentation at all. |
out of scope for this issue, yeah, but we do need to address that as part of the DSA work (for appeals and overrides) |
@wagnerand this landed with #9326 fwiw - now enable and disable are reversible. |
it's because the addon history only shows activity logs linked to a version, and the force_enable activity doesn't have an associated version (the force disable action gets linked to whatever is the current or latest version at the time of the disable). If @wagnerand has opinions on this I can address in a follow-up. |
Old Jira Ticket: https://mozilla-hub.atlassian.net/browse/ADDSRV-544 |
The existing force-disable button for add-ons needs to be moved into the reviewer actions, so that a reviewer can select a reason and a write a response or use a canned one.
This means that for the force-disable action, the reason and the response are mandatory fields.
┆Issue is synchronized with this Jira Task
The text was updated successfully, but these errors were encountered: