-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(GuildMemberManager): allow moving members to any non-text channel #5681
Conversation
Technically, this way category channels are going to be an issue, since they return |
Touché. I could change this to just add |
would an isVoice method be needed? |
Store channels will also be an issue. |
Well, yes, i wasn't really trying to provide an extensive list of every single channel type this check will not work with, but simply give an example of why And my mention of |
Since all applicable voice channels (dm channels technically have voice, but they don't count and not for bots anyways) extend |
I'll try that out tomorrow. Now that I have you all here, should I rather make a new commit or amend the existing when it's such small changes? I've heard many different opinions from other repos. |
Tested with |
change check on channel type to allow stage channel as well close discordjs#5673
Please describe the changes this PR makes and why it should be merged:
This will allow moving members to channels with type
stage
. Closes #5673 🙂Status and versioning classification: