Skip to content

Fix comment reports minds#4431

Ben requested to merge fix/comment-reports-m4431 into master

Ticket(s) / Related Merge Requests

minds#4431

Summary of Changes

Fixes issue preventing comment reports from being accepted.

Testing Considerations

This task was primarily focused around making the spam option work for comments again however whilst I was in there I fixed two additional bugs I found with Illegal: Extortion and NSFW - nsfw comments aren't fully supported BUT the option was throwing an error, which it should not have been.

Notes

  • Actions are not reflected until refresh (e.g. comments are not popped on deletion) - this has always been the case and does not relate to these changes.
  • Log in as admin using your VPN.
  • Reporting as an admin and accepting in the queue is the same thing - it hits the same endpoint.
  • You will need to unban and log back in after banning.

Checklist

  • Test reporting comments for spam.
  • Test reporting comments for extortion.
  • Test reporting comments for nsfw (should not error).
  • Test reporting comments for various other reasons.
  • Test reporting various entities for various reasons.

Deployment Considerations

Standard deployment.

Regression Scope

Report system.

Platform Affected (web, mobile, etc)

Web.

Developer Testing Completed

Manual, unit (phpspec).

Screenshots / Screen Recording

simplescreenrecorder-2023-12-12_09.23.21

simplescreenrecorder-2023-12-12_09.58.48

simplescreenrecorder-2023-12-12_10.28.48

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Ben

Merge request reports