Skip to content
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

Magmastream #542

Merged
merged 25 commits into from
Sep 9, 2023
Merged

Magmastream #542

merged 25 commits into from
Sep 9, 2023

Conversation

thororen1234
Copy link
Contributor

instead of erela.js which has been discontinued from the looks of it I moved the audio/music player to magmastream, upgraded lavalink so it would work with magmastream added the spotify plugin for lavalink so it uses lavalink instead of the bot, and added back 247 also add back the radio file that was in assets or delete radio lol it won't work now.

@Spiderjockey02
Copy link
Owner

LGTM for now. The radio command is indeed bugged, it should be fetching from my API not a non-existent JSON file, so I'll fix that now.

@thororen1234
Copy link
Contributor Author

Okay once that's done I'll readd radio

I added lavacord instead of magmastream here
@Spiderjockey02
Copy link
Owner

yep radio cmd is now fixed so u can resolve the conflict

@thororen1234
Copy link
Contributor Author

Alright I added it back

@thororen1234
Copy link
Contributor Author

Heads up I made it discordjs v14.12. Whatever comes after that I forgot and got effects you have to use the reset function Im leaving it up to you to see if you can fix that.

@Spiderjockey02
Copy link
Owner

Just had a look at lavasrc, I don't see why everything shouldn't be enabled by default?

Repository owner deleted a comment from birongliu Sep 9, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@Spiderjockey02 Spiderjockey02 merged commit fe1c1e6 into Spiderjockey02:master Sep 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants