Add flags to highlight reports on pledge taker getting reported multiple times / Adds list of other reports on the same user in report review page#46
Open
wninohira wants to merge 13 commits intoAnyKeyOrg:masterfrom
Conversation
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.
…kes small change in reports staff view
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
… Waka and PR #46 for getting this idea underway.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.
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 )
Implementation note
twitch_idto report modeltwitch_idto report modelreports_countto pledge modelreports_countbased on the existing reportsNote for merging