search

ayn2op / discordo

A lightweight, secure, and feature-rich Discord terminal client.
MIT License
2.25k stars 70 forks source link

Proper msg caching #443

Closed Retropaint closed 1 month ago

Retropaint commented 1 month ago

This PR replaces the default 100 cabinet message limit imposed by Arikawa, to the message limit specified in the user config. This prevents loading more messages than necessary since Cabinet retrieves from API if things are missing, wasting half of the messages being sent if using the default config (of 50 messages).

Somewhat unrelated but still applies: also added a limit of 100 to MessagesLimit

Retropaint commented 1 month ago

Since I'm considering #444, this issue should be resolved along with it.