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

Spotify plugin #138

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

BlueManCZ
Copy link

This is a rewrite of the old and abandoned Spotify extension.

Spotify plugin allows user to search and play tracks, add them to the queue and select a client.

@martinengelke
Copy link

Maybe we could be more general and also include e.g. Soundcloud (I like independent music), so it could be preferable a music player extension? At least for macOS it could just work with the built-in player, which is most of the time fine enough for me.

@BlueManCZ
Copy link
Author

That's an interesting idea. I was considering controlling other platforms as well (e.g., Tidal, Roon). However, if you scroll through the code, you'll notice that much of it (spotifyApiClient.cpp, configwidget.ui) is highly platform-specific, which limits what we can do. While it might be possible to generalize this into an overgrown plugin with different APIs for various services, it would be much easier to handle this as separate plugins, allowing users to choose the ones they want to use.

it could just work with the built-in player

I don't think the built-in SoundCloud player allows programmatic searching for tracks, which is the main goal of this extension. Implementing SoundCloud's /tracks API endpoint would likely be required to achieve this.

@martinengelke
Copy link

just an idea... :)
did not deep into it enough, I thought the API Requests could look or processed quite similarly and we can built a basic music player, capable of more then one music service?

@BlueManCZ BlueManCZ mentioned this pull request Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants