-
-
Notifications
You must be signed in to change notification settings - Fork 734
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
First pass on stage channels #1575
Conversation
/** | ||
* A {@link StageChannel StageChannel}, Guild-Only. | ||
*/ | ||
STAGE(13, 1, true), |
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.
Reminder: we have to update all usages of channel type in the library for this
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.
Assuming this was handled?
Freezing this for now due to discord/discord-api-docs#2751
|
Co-authored-by: caneleex <[email protected]>
The API for stage channels is kinda weird. I did some digging and found out a few issues that will be annoying to deal with:
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
It will be ready when it is ready. Stage channels are still in flux behind the scenes in the Discord API. |
/** | ||
* A {@link StageChannel StageChannel}, Guild-Only. | ||
*/ | ||
STAGE(13, 1, true), |
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.
Assuming this was handled?
/** | ||
* Create a new {@link StageInstance} for this stage channel. | ||
* | ||
* <p>If there already is an active {@link StageInstance} for this channel, this will fail with {@link net.dv8tion.jda.api.requests.ErrorResponse#STAGE_ALREADY_OPEN ErrorResponse.STAGE_ALREADY_OPEN}. |
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.
Is it possible to do pre-checks to see if the stage is active?
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.
I wanted to avoid doing pre-checks to avoid race-conditions when recreating a stage instance.
src/main/java/net/dv8tion/jda/internal/requests/restaction/ChannelActionImpl.java
Show resolved
Hide resolved
src/main/java/net/dv8tion/jda/internal/requests/restaction/StageInstanceActionImpl.java
Show resolved
Hide resolved
src/main/java/net/dv8tion/jda/internal/handle/VoiceStateUpdateHandler.java
Show resolved
Hide resolved
* Add support for speaker moderation * Add GuildVoiceState#inviteSpeaker/approveSpeaker/declineSpeaker * Add check for missing members in voice states * Add support for stage instances * Add check for missing member in voice state update * Properly remove lurkers from cache * Add audit log enum constants
* Add support for speaker moderation * Add GuildVoiceState#inviteSpeaker/approveSpeaker/declineSpeaker * Add check for missing members in voice states * Add support for stage instances * Add check for missing member in voice state update * Properly remove lurkers from cache * Add audit log enum constants
* Add support for speaker moderation * Add GuildVoiceState#inviteSpeaker/approveSpeaker/declineSpeaker * Add check for missing members in voice states * Add support for stage instances * Add check for missing member in voice state update * Properly remove lurkers from cache * Add audit log enum constants
Pull Request Etiquette
Changes
Closes Issue: #1572
Description
This adds support for stage channels, which are basically just voice channels with topics.