Hypixel-Translators / hypixel-translators-bot

Welcome to the Hypixel Translators Bot GitHub repository! Here you can report bugs, request features and contribute to the code.
https://discord.gg/rcT948A
MIT License
23 stars 11 forks source link

Cooldown on /hypixelverify #743

Closed TreasonMay closed 2 years ago

TreasonMay commented 2 years ago

Feature description

Cooldown is way too long - have to wait after being told what to do to verify, in order to verify.

Alternative solutions

Additional context

I wrote all of this during this cooldown, that's just how long it is.

ImRodry commented 2 years ago

The cooldown of 10 minutes on the hypixelverify command is intentional and will not change. This is because the API we use to fetch the data from Hypixel caches your data for 10 minutes after it was accessed, so if you change anything within that timeframe the bot wouldn’t know because the data would be outdated.

TreasonMay commented 2 years ago

Here's a suggestion then: Use the official, uncached, Hypixel API just for this feature.

ImRodry commented 2 years ago

That is something I can consider since we don’t need any of the data transformations that slothpixel does, however we’d need to be careful not to hit ratelimits. You should open a different issue if you want to keep that tracked but I may end up doing it either way.