Skip to content

Conversation

@wninohira
Copy link

Summary

This pull request addresses parts of #20, and allows moderator to see which reports are about a user that has gotten reported multiple times (= more urgent reports). Implementation is based on the moderation experience of the IDL lab group and the discussion among us. It has two main interface changes in report list view and report detail view.
*This is not ready to get merged into the live codebase until the rake task to look up twitch-id for all existing reports is ready.

Report list view

An icon to show that the user has been reported multiple times and the number is displayed next to their twitch username.

drawing

Report detail view

Added "other reports" section to show other reports about the same user. It looks up user based on the twitch id. It also shows the status of the reports (Unsolved / warned / dismissed / revoked )

drawing

Implementation note

  • Adding twitch_id to report model
    • adds column twitch_id to report model
    • Updates report reception process so that twitch_id for the reported username gets looked up and added to data
    • TODO: make a rake task that looks up twitch_id for existing reports
  • Adding association between report and pledge
    • Adds column reports_count to pledge model
    • makes a rake task that resets the initial value for reports_count based on the existing reports

Note for merging

  1. Run migration
  2. run twitch_id lookup rake task (not in this PR)
  3. run reports_count resetter rake task

wninohira and others added 13 commits April 19, 2022 14:53
Updates find_reported_twitch_user function in both warnings (where it…
…additional changes are required for this implementation to work (looking up twitch ID as a report is submitted & rake task to go through existing reports and looks up twitch ID)
… on the reported username. Adds rake task that performs the same task on all existing reports.
… replaces old variable reported_twitch_user with report.twitch_id
… report index view. Adds assocications between report and pledge model with twitch_id as a key to anable counting of the reports.
becuase it needs to be changed with the consideration for twitch API limit.
…orts_counter.rake task to not count reports where twitch_id is empty
deadroxy added a commit that referenced this pull request Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant