-
Notifications
You must be signed in to change notification settings - Fork 580
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
Implement VOICE_CHANNEL_STATUS_UPDATE #2565
Conversation
62fb38e
to
9a6a135
Compare
I have gone ahead and pushed a new function to EditChannel to change the status, not sure if the comments I've put on it are 100% correct because I am yet to fully test. Also included the permission to do so, not sure if I need to change anything else alongside that. I am pulling everything off discord/discord-api-docs/pull/6398 but its not merged yet so things might change or I might be wrong in some places. |
If anything would be better off split into a different pull request I'll do so, just ask. |
c3f12f1
to
4339118
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doing this for a workshop sorry
I will cleanup the code, format this right later and document whatever I can later. I thought thought it would be in a decent enough state to be opened as a draft.
Not sure if you even want this yet given that the documentation for this isn't even merged yet and is just a draft pull request on discord/discord-api-docs
I had no idea what I was doing with the CacheUpdate, so the code is probably not even close to what it could be.