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

Traffic Improvement #4

Open
G3rrus opened this issue Apr 30, 2021 · 0 comments
Open

Traffic Improvement #4

G3rrus opened this issue Apr 30, 2021 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@G3rrus
Copy link
Owner

G3rrus commented Apr 30, 2021

At the moment each function itself is responsible for sending out the messages back to discord. Since initially I didn't plan making a command which returned a list of all commands and how to use them, this was no problem. However now that I have it the way it is, it spams the chat with multiple messages (like one command each message). Changes could be implemented where each function only returns String and I make an additional function which is responsible for basically sending that String "out". However the one concern that this leaves and why in the end the way I'm doing it now might not be bad is that there is a max character limit of 2000 so if I ever expand on the commands the 'command-list' command might run out of characters and fail or send a .txt instead (behavior unclear). The info could maybe be packed into an embed though I have no idea what sort of restrictions apply there.

@G3rrus G3rrus added the enhancement New feature or request label Apr 30, 2021
@G3rrus G3rrus added this to the 1.0.1 milestone Apr 30, 2021
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
None yet
Development

No branches or pull requests

1 participant