Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Fixed the manually disconnect issue. #34

Closed
wants to merge 1 commit into from

Conversation

Ariz-z
Copy link
Contributor

@Ariz-z Ariz-z commented Feb 8, 2022

If the music is playing and any of the mod/admin disconnect the bot from the voice channel from right click then if you try to play the song again it won't work. But now it will work properly i have tested this

If the music is playing and any of the mod/admin disconnect the bot from the voice channel from right click then if you try to play the song again it won't work. But now it will work properly i have tested this
@@ -246,6 +254,7 @@ async def _on_voice_state_update(self, member, before, after):

await self.cleanup(voice_client, member.guild)


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an unnecessary new line

@@ -233,6 +233,14 @@ async def _on_voice_state_update(self, member, before, after):
voice_client = member.guild.voice_client
channel_change = before.channel != after.channel

if (before.channel is not None) and (after.channel is None):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be shorted down to

if before.channel and not after.channel:

@adam757521
Copy link
Member

I've left some changes, please look at them and commit them.

@adam757521
Copy link
Member

Closed due to inactivity.

@adam757521 adam757521 closed this Mar 12, 2022
@Ariz-z Ariz-z deleted the patch-3 branch March 12, 2022 19:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants