Skip to content

[front] Tenant report system #2664 minds#4368

Ben requested to merge feat/report-v2-e2664 into master

Ticket(s) / Related Merge Requests

Summary of Changes

Front-end for tenant report panel in moderation queue. Does NOT include report creation.

Testing Considerations

Use https://c81e728d9d4c2f636f067f89cc14862c.bens-networks-3.oke.minds.io

Notes

  • Be sure to read engine card as that has different testing considerations.
  • To add new reports you will need to make the GraphQL call WITH the correct entity URNs - because of this PLAN YOUR TEST PLAN CAREFULLY or you will end up having to create a bunch of reports, which isn't trivial.
  • Changes to design (no toolbar / menu etc) have been signed off by @chints98.

Checklist

  • List paginates.

  • User report: Ban action

  • User report: Ignore action

  • Group report: Delete action

  • Group report: Ignore action

  • Activity report: Ban action

  • Activity report: Delete action

  • Activity report: Ignore action

Note: comments are currently broken on sandbox so the below have been tested locally
  • Comment report: Ban action
  • Comment report: Delete action
  • Comment report: Ignore action

Deployment Considerations

Standard deployment.

Regression Scope

Mostly additive changes to network admin panel moderation tab.

Platform Affected (web, mobile, etc)

Web.

Developer Testing Completed

Karma/jasmine and manual.

Screenshots / Screen Recording

Ban user reports (ignore broken report - fixed in another push)

simplescreenrecorder-2023-11-14_11.44.31

Ignore user reports

simplescreenrecorder-2023-11-14_13.08.35

Group ignore / delete

simplescreenrecorder-2023-11-14_13.10.27

Activity ignore / delete / ban

simplescreenrecorder-2023-11-14_13.14.42

Ban/ignore/delete comment reports (TESTED LOCALLY)

simplescreenrecorder-2023-11-14_12.32.59

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