Shoukaku
Shoukaku copied to clipboard
Add lavalink state (useful for resuming system)
Typings are not correctly specified, check github actions for info
Any benefit of this?
allows to know if the bot has been disconnected during a restart
I dont get what you mean, this pr seems useless
if you don't have a system to resume the players then it will be useless for you
No i mean, this is not used internally by shoukaku it self. why dont you just extend the player instead?
because it could be useful for other people and it doesn't impact the good functioning of shoukaku
Instead of playerState, I'd call it connected, and do not make it type boolean|nulll
instead make it only a boolean type that defaults to false until LL sends the first playerUpdate to this player. playerState = true
is unclear on what it provides / says
Could also be our own type, eg: type state = Connected | Disconnected | Unknown
Closing due to inactivity, re-open if you applied the changes I said above