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

feat!: move options to options field and add command-specific options #138

Open
NanderTGA opened this issue Dec 16, 2023 · 0 comments
Open
Labels
enhancement New feature or request

Comments

@NanderTGA
Copy link
Owner

NanderTGA commented Dec 16, 2023

Currently, every option is a field on the Client class.
The new approach will have all the options in a separate field.
For example: client.helpCommandLimit
New approach: client.options.helpCommandLimit
I'm changing it like this for more clarity.

I also want to add a feature where you can override certain options for a specific command.
I'll add a new field to Command and one to CommandContext to change and view the options respectively.

And while we're at it, why not let an option be a function that takes the context and any more arguments that provide more information (or maybe put them on the context too?)

@NanderTGA NanderTGA added the enhancement New feature or request label Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo
Development

No branches or pull requests

1 participant