Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

make !reconnect have the bot leave+reconnect, instead of just reconnecting #31

Open
qixils opened this issue Mar 4, 2018 · 8 comments
Labels

Comments

@qixils
Copy link
Member

qixils commented Mar 4, 2018

basically turn !reconnect's current functionality into what you would get from doing !disconnect, followed by !reconnect

@Bottersnike
Copy link
Contributor

That's what it already does? No?

@Tantusar
Copy link
Contributor

Tantusar commented Mar 4, 2018

No, that is not what it already does. It just says, "Already connected, sucks to be you." Okay, it's a little more polite.

@Bottersnike
Copy link
Contributor

I'm pretty sure I patched that.

@Bottersnike
Copy link
Contributor

For context:

if vc is not None:
it locates any voice channel it's connected to and force-disconnects.

@Bottersnike
Copy link
Contributor

Ensured fixed with 1456219 Please reopen if continuing.

@qixils
Copy link
Member Author

qixils commented Apr 2, 2018

image

@qixils qixils reopened this Apr 2, 2018
@Bottersnike
Copy link
Contributor

Right, so,

await ctx.voice_client.disconnect()
ensures that if it has a reference to the voice channel anywhere, it disconnects. The problem would appear to be that for some reason it no longer has a ref.

@qixils qixils added bug and removed enhancement labels Apr 6, 2018
@qixils
Copy link
Member Author

qixils commented Aug 2, 2018

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants