allthingslinux / tux

Tux is an all in one bot for the All Things Linux discord server.
https://discord.gg/linux
GNU General Public License v3.0
85 stars 25 forks source link

[FEATURE] - llb (level leader board command) #600

Open cherryblogsmedia opened 1 month ago

cherryblogsmedia commented 1 month ago

Is your feature request related to a problem? Please describe

essentially display the top 25 or 10 members (as @electron271 mentioned) with the most xp or highest levels

Describe the solution you'd like

make it not super fancy and stuff ( like @CapnRyna mentioned) to make it less of a flex or competitive (as @kzndotsh desires)

Describe alternatives you've considered

another one could be a role like top 10 levels or top 50 such as the text and vc roles of similar nature

Additional Context

not sure if kaizen would like this or not but i think its cool idea so i put it here regardless

EternalBlissed commented 1 month ago

Meow

electron271 commented 1 month ago

@kzndotsh if you are fine with this suggestion I could implement it

Atmois commented 1 month ago

This can be implemented but there should be a toggle if servers wish to use the "leaderboard" as we in ATL have voted against it.

@electron271

cherryblogsmedia commented 1 month ago

thats a good idea actually

electron271 commented 1 month ago

This can be implemented but there should be a toggle if servers wish to use the "leaderboard" as we in ATL have voted against it.

@electron271

just disable the cog in .env

kzndotsh commented 1 month ago

@kzndotsh if you are fine with this suggestion I could implement it

does it matter now that we have capped it?

electron271 commented 1 month ago

@kzndotsh if you are fine with this suggestion I could implement it

does it matter now that we have capped it?

yes for other servers, tux is no longer atl specific

just make it in a separate cog so you can just disable that cog in dotenv

kzndotsh commented 1 month ago

@kzndotsh if you are fine with this suggestion I could implement it

does it matter now that we have capped it?

yes for other servers, tux is no longer atl specific

just make it in a separate cog so you can just disable that cog in dotenv

Go for it