Reset all sessions on ready state

pinano-discord / Pinano-Discord-Bot

A Discord bot for the Pinano guild

MIT License

9 stars 11 forks source link

Reset all sessions on ready state #97

Closed pianiststickman closed 5 years ago

pianiststickman commented 5 years ago

We have some non-deterministic behaviour caused by cases where connectivity issues impact our ability to receive voiceStateChanged events. Rather than try to figure out what may or may not have happened in the intervening time, clear the session instead and treat it as lost.

pianiststickman commented 5 years ago

Additional bonus: if we reset all sessions on ready state, this means that we don't need to mute/unmute all the active users every time we reset the bot.

pianiststickman commented 5 years ago

Fixed in #108. Note that if discord.js gives us bad data we're still hooped, but we can restart anytime.