-
Notifications
You must be signed in to change notification settings - Fork 4
feat: add audit for auto-reporting for duplicate user ratings #147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: beta
Are you sure you want to change the base?
Conversation
|
View your CI Pipeline Execution ↗ for commit e77df6f
☁️ Nx Cloud last updated this comment at |
Deploying polyratings with
|
| Latest commit: |
e77df6f
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://1a5ae9e8.polyratings.pages.dev |
| Branch Preview URL: | https://dedupe.polyratings.pages.dev |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot I have merged in the upstream so this PR's diff is more relevant. Given this update, please analyze which comments are still relevant to the core PR, and fix them in a new PR that merges into the dedupe branch |
|
@cjlawson02 I've opened a new pull request, #155, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@mfish33 ready for re-review |
* feat: switch to OpenAI omni-moderation model * fix: admin score presentation backwards compatibility * fix: less pollution in analyzed scores * fix: address some comments * fix: address other comments * fix: lint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2ba3871 to
960451d
Compare
This pull request adds an automated audit feature to help administrators detect and report duplicate user ratings. The main changes include a new backend mutation to scan for duplicate ratings based on anonymous identifiers and a corresponding frontend button to trigger this audit from the admin interface.
Audit feature implementation:
autoReportDuplicateUsersto theadminRouterinadmin.ts. This mutation scans all professors' reviews for duplicate ratings submitted by the same anonymous user and automatically creates reports for each duplicate.Frontend integration:
Admin.tsxto call theautoReportDuplicateUsersbackend mutation and refresh the reports list on success.