-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: Mod Actions Panel #136
Conversation
Here's the code health analysis summary for commits Analysis Summary
|
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.
✅ Code Health Quality Gates: OK
Change in average Code Health of affected files: +0.39 (9.49 -> 9.88)
- Improving Code Health: 2 findings(s) ✅
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.
✅ Code Health Quality Gates: OK
Change in average Code Health of affected files: +0.22 (9.41 -> 9.63)
- Improving Code Health: 3 findings(s) ✅
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.
✅ Code Health Quality Gates: OK
Change in average Code Health of affected files: +0.22 (9.41 -> 9.63)
- Improving Code Health: 3 findings(s) ✅
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.
✅ Code Health Quality Gates: OK
Change in average Code Health of affected files: +0.04 (9.42 -> 9.47)
- Declining Code Health: 2 findings(s) 🚩
- Improving Code Health: 3 findings(s) ✅
@@ -144,4 +144,32 @@ export default class UserDbManager extends BaseBlacklistManager<userData> { | |||
|
|||
await opts.target.send({ embeds: [embed] }).catch(() => null); | |||
} | |||
|
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.
❌ New issue: Primitive Obsession
In this module, 47.8% of all function arguments are primitive types, threshold = 30.0%
@@ -144,4 +144,32 @@ export default class UserDbManager extends BaseBlacklistManager<userData> { | |||
|
|||
await opts.target.send({ embeds: [embed] }).catch(() => null); | |||
} | |||
|
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.
❌ New issue: String Heavy Function Arguments
In this module, 47.8% of all arguments to its 13 functions are strings. The threshold for string arguments is 39.0%
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.
✅ Code Health Quality Gates: OK
Change in average Code Health of affected files: +0.04 (9.42 -> 9.47)
- Declining Code Health: 2 findings(s) 🚩
- Improving Code Health: 3 findings(s) ✅
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.
✅ Code Health Quality Gates: OK
Change in average Code Health of affected files: +0.12 (9.50 -> 9.61)
- Declining Code Health: 2 findings(s) 🚩
- Improving Code Health: 4 findings(s) ✅
0e0196c
to
ad56988
Compare
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.
✅ Code Health Quality Gates: OK
Change in average Code Health of affected files: +0.12 (9.50 -> 9.62)
- Declining Code Health: 2 findings(s) 🚩
- Improving Code Health: 4 findings(s) ✅
Quality Gate passedIssues Measures |
Moderator Actions Context Menu
Display a panel for moderator actions context menu command, including:
Message Info Button (too hard)Remove reactions (do this via dashboard plskthx)This can be made available as a button attached to reports, message info ctx menu, and as a separate "Moderator Actions" ctx menu command.
Improved
Message Info/Report
Context Menu for ModerationThere should be a drop-down that seperates the basic info and the advanced info (if you're a hub moderator): When you select the advanced info, it has buttons to blacklist the user (who sent the message) and the server (where the message is from).
Instead of the usual info in the embeds, replace it with information of the user's "Recent Offenses" so that we can make sure if we should lengthen their next blacklist duration.
Suggested by:
whos.eth4n
Resolves #124
Resolves #111