Florian Spieß
Florian Spieß
Is this rewrite going to happen on an existing branch or are you going to add a rewrite branch?
I can only offer JDA as an example: - [AudioNatives](https://github.com/DV8FromTheWorld/JDA/blob/master/src/main/java/net/dv8tion/jda/api/audio/AudioNatives.java) - [Decoder](https://github.com/DV8FromTheWorld/JDA/blob/master/src/main/java/net/dv8tion/jda/internal/audio/Decoder.java) - [AudioConnection](https://github.com/DV8FromTheWorld/JDA/blob/6d5db0f98dfd5c1a0fb72864af67e0b56affd13a/src/main/java/net/dv8tion/jda/internal/audio/AudioConnection.java#L742)
Why is this PR still a draft? Are these changes not happening anymore?
What is holding this back?
How long is this feature going to be in beta? Should libraries add support or not?
Has this feature been forgotten? @shaydewael what's blocking this?
What is holding this back?
> Is there anything preventing this from being merged now? I want to see if I can get a test bot to test the changes myself too, other than that...
Looks like there is still a compilation issue here with the constructor usage. Have you tested whether the usage of [`with_expiration=true`](https://discord.com/developers/docs/resources/invite#get-invite-query-string-params) makes a difference? It doesn't seem like we use...
Gonna hold off on merging this, still waiting for some info regarding the API stability here.