Skip to content
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

Merged
merged 4 commits into from
Oct 20, 2023

Conversation

jamesbt365
Copy link
Member

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.

@github-actions github-actions bot added model Related to the `model` module. client Related to the `client` module. cache Related to the `cache`-feature. labels Oct 16, 2023
@arqunis arqunis added enhancement An improvement to Serenity. discord feature Related to Discord's functionality. labels Oct 16, 2023
@jamesbt365 jamesbt365 marked this pull request as ready for review October 18, 2023 11:23
@github-actions github-actions bot added builder Related to the `builder` module. command_attr Related to the `command_attr` crate. labels Oct 18, 2023
@jamesbt365
Copy link
Member Author

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.

@jamesbt365
Copy link
Member Author

If anything would be better off split into a different pull request I'll do so, just ask.

Copy link
Contributor

@AngelOnFira AngelOnFira left a 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

@arqunis arqunis merged commit 136330a into serenity-rs:next Oct 20, 2023
21 checks passed
@jamesbt365 jamesbt365 deleted the voice-channel-status branch October 20, 2023 12:44
mkrasnitski pushed a commit to mkrasnitski/serenity that referenced this pull request Oct 24, 2023
arqunis pushed a commit to arqunis/serenity that referenced this pull request Oct 24, 2023
arqunis pushed a commit to arqunis/serenity that referenced this pull request Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builder Related to the `builder` module. cache Related to the `cache`-feature. client Related to the `client` module. command_attr Related to the `command_attr` crate. discord feature Related to Discord's functionality. enhancement An improvement to Serenity. model Related to the `model` module.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants