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

Add support for Piped links #1556

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Yohr011
Copy link

@Yohr011 Yohr011 commented Apr 26, 2024

This pull request...

  • Fixes a bug
  • Introduces a new feature
  • Improves an existing feature
  • Boosts code quality or performance

Description

Allows users to send Piped links to the music bot

Purpose

Users who prefer to use Piped over YouTube can send their instance's links to the bot

Relevant Issue(s)

This PR closes issue #1492

Added a command to set the Piped instance URL. Also added Piped to the config file.
Bot will accept music commands using the user's Piped instance links.
@IngwiePhoenix
Copy link

Dude! O.O This is so cool! Thanks so much for the work - this is awesome!

@jagrosh
Copy link
Owner

jagrosh commented Apr 26, 2024

This should probably be implemented as either a source manager or a transform

uncreativeCultist added a commit to uncreativeCultist/MusicBotShiverFork that referenced this pull request Jul 11, 2024
Hopefully would result in better performance in different regions.
@Binzy-Boi
Copy link

Sorry if this isn't where I would put this comment or if it's best suited as a new issue, but since they both have the same function, similar popularity, and open API documentation, this might be a good opportunity to also add support for Invidious links.

Documentation can be found here: https://docs.invidious.io/api/

Appreciate the work you've put into this PR.

@jagrosh
Copy link
Owner

jagrosh commented Aug 6, 2024

New source requests should be made to lavaplayer or implemented in a standalone source manager; this is not something that JMusicBot would be providing exclusively.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants