-
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
Codeycoin leaderboard #263
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.
I do not think that bots should be able to have Codey coins, and therefore bots should definitely not appear on the leaderboard. While this change might seem to be out of scope of this PR, I think it would be nice to include this change since the leaderboard is client-facing.
Also, it looks like there are some linter warnings.
while u are here |
ce334c9
to
914d3a2
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.
With regard to my previous comment about bots, I made a new issue, #265.
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.
high demand feature :) tyty
made some tiny notes
@@ -61,6 +61,11 @@ export const coinBonusMap = new Map<BonusType, Bonus>([ | |||
] | |||
]); | |||
|
|||
export interface UserCoinEntry { |
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.
I can't ctrl f on phone but can u just lmk where this is used XD
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 the leaderboard :))
Summary of Changes
Added a leaderboard for the top 10 CodeyCoin holders.
Steps to Reproduce
Run
.coin leaderboard
or/coin leaderboard
.Screenshots
The embed is the same regardless whether you use regular or slash commands.