Skip to content
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

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

Open
Icosicherrys opened this issue Oct 5, 2024 · 8 comments
Open

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

Icosicherrys opened this issue Oct 5, 2024 · 8 comments

Comments

@Icosicherrys
Copy link
Contributor

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
Copy link

Meow

@electron271
Copy link
Contributor

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

@Atmois
Copy link
Contributor

Atmois commented Oct 5, 2024

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

@Icosicherrys
Copy link
Contributor Author

thats a good idea actually

@electron271
Copy link
Contributor

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
Copy link
Contributor

kzndotsh commented Oct 6, 2024

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

does it matter now that we have capped it?

@electron271
Copy link
Contributor

@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
Copy link
Contributor

kzndotsh commented Oct 6, 2024

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

5 participants