Adapt to upcoming breaking API change in Lavalink for DAVE#42
Adapt to upcoming breaking API change in Lavalink for DAVE#42freyacodes wants to merge 1 commit into
Conversation
| val update = event.current | ||
| if (update.userId != update.client.selfId) return@on Mono.empty() | ||
| val channel = update.channelId.getOrNull() | ||
| channelId.set(channel?.asString()) |
There was a problem hiding this comment.
unless we can guarantee the order of these events I have doubts about this working
There was a problem hiding this comment.
Yeah me too, but it was worth trying. Anyways, I'm going to close this PR since #43 looks more polished
|
I pulled your changes into #43 and made a few tweaks to ensure it handles multi-guild support properly. That's why I'm closing this branch in favour of the new one (totally didn't notice you already made one) |
|
I see yeah. It's customary to write something like |
I am so sorry I forgot that yesterday, rebased the history in that branch to include you. Thanks for the reminder <3 |
Not tested yet, but could be used together with lavalink-devs/Lavalink#1190