-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update the docs for speaking updates #5576
Conversation
@@ -234,6 +234,9 @@ The following flags can be used as a bitwise mask. For example `5` would be prio | |||
> warn | |||
> You must send at least one [Opcode 5 Speaking](#DOCS_TOPICS_OPCODES_AND_STATUS_CODES/voice) payload before sending voice data, or you will be disconnected with an invalid SSRC error. | |||
|
|||
> info | |||
> The `delay` property is ignored for voice connections via the voice gateway. | |||
|
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.
Not true and it is very important. For browser, it must be set to VAD leading value. For Discord Client, it must be set to zero.
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.
This is only usable by bots though, and bots can't be used in a browser context due to CORS. I'm not sure what I should write then?
I've changed it to just say that bots should set it to 0.
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.
Sounds good.
|
||
The following flags can be used as a bitwise mask. For example `5` would be priority and voice. | ||
The following flags can be used as a bitwise mask. For example `5` would be priority and voice. The speaking mode should not be 0 to allow sending audio. |
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.
Do not understand why this sentence is needed.
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.
Because you should never set it to 0, it only causes issues.
These docs are quite outdated, and you no longer need to send a speaking stop indication. The discord client now handles this automatically by removing the speaking indicator when the UDP packets stop arriving.