Skip to content

Commit

Permalink
Remove pre-existing bulk update from report model save methods
Browse files Browse the repository at this point in the history
  • Loading branch information
sarayourfriend committed Dec 18, 2023
1 parent aba1c7b commit b41b487
Showing 1 changed file with 5 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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

Expand Down

0 comments on commit b41b487

Please sign in to comment.