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

Refactor: Multi-Messages #14

Open
p5quared opened this issue Mar 17, 2023 · 0 comments
Open

Refactor: Multi-Messages #14

p5quared opened this issue Mar 17, 2023 · 0 comments
Labels
good first issue Good for newcomers

Comments

@p5quared
Copy link
Owner

There are many instances in the program where we want to send multiple lines of text to users. Often, we are doing this with multiple calls to ctx.send('asdf') or similar. I am pretty sure this is a bad practice because it is splitting up what should be one server request into several.

Most if not all instances where we are using ctx.send() multiple times in a row should be replaced by ctx.send("line1\nline2\n...etc")

@p5quared p5quared added the good first issue Good for newcomers label Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant