-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Adds support for JDA 5 / Modify the Lavaplayer lib #1447
base: master
Are you sure you want to change the base?
Conversation
Important to mention that this PR also switches to a jda-utilities fork and a lavaplayer fork. jagrosh has stated that he's been working on adding slash commands to jda-utilities. As for lavaplayer, it seems likely to me that we will switch to a fork though. |
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'd wager that right now, it's too soon to update to JDA 5. Regardless, I imagine this PR will be nice to start off with once jda-utils gets updated, so we can quickly make the move to JDA 5 on MusicBot, to then start work on slash commands & i18n.
Hello ! I don't explicitly mention the change to another Librairy Lavaplayer. I apologise for this. As far as JDA Utilities is concerned, I am indeed using a fork, but as specified in the pom.xml, this is temporary until Jagrosh accepts the pull request on the JDA-Utilities Github or recodes JDA-Utilities to make it work under JDA5. I've cancelled my changes to the pom.xml, I think there should still be a comment about JDA-Utilities, if it's disturbing I'll delete it. The compiler produces an obvious error because JDA5 is not supported by JDA-Utilities. Apart from that, I sincerely thank you for the time you spend maintaining this project. If there is anything I can do to help, I am willing to do so. Sorry for my English which can be a bit disastrous in some aspects, I mainly speak French. (Thanks DeepL) |
Hey, I experienced the bug which is mentioned to be fixed with this PR so I tried to build it using the GitHub action, but it failed. I'm not sure on how to fix this, it seems to fail to download some dependencies. EDIT: I just noticed that these are the same errors as in the check bellow which fails... |
As requested by MichailiK, I've modified the .pom by putting back the old imports. You can only build the first commit, not the last. |
Alright, thank you for the quick response! I'll build with the previous commit in that case. |
I've compiled it using the second to last commit from MasqueOu's fork. This is how I achieved it:
I must note though, in my case, this did not fix the issue it says it should. |
Hello ! EDIT: I've tested this library at considerable length, as well as the one that is included as standard in the pull request. I never got a 403 error again. The new version may correct the problem, but I can't be sure. Don't hesitate to give your feedback. |
After messing around with the code for along time I finally managed to make it compile. I made changes to the pom.xml file, that fixed the 403 error. (might not be a final fix, but it solved my problems at least) Source code and compiled binary available here: https://github.com/IIWDTI/MusicBot |
This pull request...
Description
This pull-request supports JDA 5.0 and modifies certain dependencies in order to fix the problems already listed.
Purpose
In the near future, this will enable us to use the full potential of Discord's latest features, including support for Threads, new usernames, Slash commands, etc.
Relevant Issue(s)
This PR closes issue #1434, #1437