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

Server Quotes -- !allquotes bottleneck mitigation #123

Open
kringeth000 opened this issue Jun 17, 2020 · 0 comments
Open

Server Quotes -- !allquotes bottleneck mitigation #123

kringeth000 opened this issue Jun 17, 2020 · 0 comments
Labels
enhancement Reworking/Tweaking of an existing feature

Comments

@kringeth000
Copy link
Member

Due to rate limiting, using !allquotes sends ~5 messages at a time and gets rate limited sending individual quotes as individual messages to the recipient; this can take at least a minute on servers with 100+ quotes. Possibly could be mitigated by making a pagination with the message to navigate through pages, or sending a link either to the channel the command is issued in. This could also be sent to the requestor via DM to view the quotes from that server (or more in the future) via a web browser. This could either be just a generic link that works for showing the server quotes, or by use of a token in the URL.

Not a breaking issue (currently) but could be a bottleneck in the future as the database and number of servers grow.

[ Could make use of features in #122 ]

@kringeth000 kringeth000 added the enhancement Reworking/Tweaking of an existing feature label Jun 17, 2020
@kringeth000 kringeth000 changed the title Server Quotes -- Web Display Server Quotes -- !allquotes bottleneck mitigation Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Reworking/Tweaking of an existing feature
Projects
None yet
Development

No branches or pull requests

1 participant