-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
(#9) added warn function #25
base: develop
Are you sure you want to change the base?
Conversation
src/commands/admin/warn.js
Outdated
return; | ||
} | ||
|
||
let data = await userService.fetchUser(interaction.user.id); |
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.
data is referring to src/database/user.service.js
and the idea in Klara was to have a reputation system that would follow all servers with the bot, getting negative reputation on warns and positive on reactions on messages. A positive warn value should be added to src/database/guild.service.js
and a reason
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.
A guild id should be added indeed, Nick should not transfer reputation over to another server.
import { SlashCommandBuilder } from '@discordjs/builders'; | ||
import { MessageEmbed, Permissions } from 'discord.js'; | ||
|
||
export const data = new SlashCommandBuilder() |
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.
Should have an option for warn reasons
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.
Will add that!
Codecov Report
@@ Coverage Diff @@
## develop #25 +/- ##
==========================================
+ Coverage 5.66% 5.75% +0.09%
==========================================
Files 20 24 +4
Lines 477 504 +27
Branches 76 80 +4
==========================================
+ Hits 27 29 +2
- Misses 395 417 +22
- Partials 55 58 +3
Continue to review full report at Codecov.
|
Basically this is transferred from Klara, but with the changes applied to this bot adviced by David.
7a9d312
to
ba9f0ae
Compare
Basically this is transferred from Klara, but with the changes
applied to this bot adviced by David.
Warn a user
Fixes #9
Motivation and Context
New functionality
How Has This Been Tested?
Tested in the development channel
Screenshots (if appropriate)
Types of changes
Checklist