-
Notifications
You must be signed in to change notification settings - Fork 18
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
Exclude bots from leaderboard #281
Conversation
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.
In terms of not having a database, shouldn't SQLite be installed due to being found in yarn.lock?
I might as well add that the change would be taking place in the messageCreate listener instead of the leaderboard, since bots shouldn't have a coin balance at all. Bots not appearing in the leaderboard would just be one effect of this. |
The issue has been updated just now for clearer specifications; sorry about that! |
No problem it's ready to test |
src/listeners/messageCreate.ts
Outdated
@@ -112,6 +113,11 @@ export class MessageCreateListener extends Listener { | |||
return; | |||
} |
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.
This case can be removed, since it is already covered below.
Also, do you mind adding 275813801792634880 as IRC_USER_ID for staging and updating README.md for the new config variable IRC_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.
LGTM!
#265
I don't have a database to test this ticket