darklab8 / fl-darkbot

discord bot to freelancer discovery community for player bases, players and forum messages tracking with alerting.
https://darklab8.github.io/fl-darkbot/
Other
4 stars 1 forks source link

allow changing time between msgs cleaning #2

Closed dd84ai closed 11 months ago

dd84ai commented 11 months ago

After some consideration, i think this feature should be rejected. Bot depends on deleting previous msgs in order to function correctly in current implementation Bot makes reliability with resending msgs through comparing them to 100 of maximum allowed to query in one request msgs from Discord channel. Allowing changing time between msgs cleansings will encourage potentially missplaced bot usage that leads to its broken behaviour.

Changing code logic in order to track msgs beyond 100 count, i don't see a point. Since a lot of code changes and potential bad performance impact for very little gain