From b41b487afa592f6520eb5627004c8cb9aec5a4bf Mon Sep 17 00:00:00 2001 From: sarayourfriend <24264157+sarayourfriend@users.noreply.github.com> Date: Mon, 18 Dec 2023 14:30:54 +1100 Subject: [PATCH] Remove pre-existing bulk update from report model save methods --- ...1208-implementation_plan_django_admin_moderator_access.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/documentation/projects/proposals/trust_and_safety/content_report_moderation/20231208-implementation_plan_django_admin_moderator_access.md b/documentation/projects/proposals/trust_and_safety/content_report_moderation/20231208-implementation_plan_django_admin_moderator_access.md index 7f530331e1b..2a220937cad 100644 --- a/documentation/projects/proposals/trust_and_safety/content_report_moderation/20231208-implementation_plan_django_admin_moderator_access.md +++ b/documentation/projects/proposals/trust_and_safety/content_report_moderation/20231208-implementation_plan_django_admin_moderator_access.md @@ -234,6 +234,10 @@ update the selected reports to set the `decision_id` to point to the new moderation decision. All actions rely on the selected set of reports and should not modify any other reports. +Remove all logic from the report models `save` methods that bulk actions +reports. That approach will no longer be valid and we will operate on specific +reports individually following the `ModerationDecision`. + ##### Multiple-decision prevention (soft-locking) To prevent multiple moderators unintentionally looking at the same report, we'll @@ -445,6 +449,7 @@ For each step description, ensure the heading includes an obvious reference to t - Create the new `ModerationDecision` models - Create an `AbstractModerationDecision` for each media type - Add new `decision_id` column to report models +- Remove bulk report updates from the report models' `save` methods ### 5. Media moderation view