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

WIP: feat(spotify): multi-user connect device #285

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

phts
Copy link
Contributor

@phts phts commented Jan 7, 2024

Screenshot 2024-01-07 at 22 38 51

Replicates functionality like v3 plugin - it allows to connect to the device for all users in the network regardless currently signed in user

@phts phts force-pushed the spotify-multiuser-device branch from f84f35e to 465b230 Compare January 7, 2024 21:44
@phts phts changed the title feat(spotify): multi-user connect device WIP: feat(spotify): multi-user connect device Jan 8, 2024
@phts
Copy link
Contributor Author

phts commented Jan 8, 2024

⚠️ Not ready yet. There is wrong behavior in some cases. Still WIP...

@phts phts marked this pull request as draft January 8, 2024 18:17
@Fastmedic
Copy link

Thank you for making this update!

@phts phts force-pushed the spotify-multiuser-device branch from 465b230 to b6172fe Compare February 9, 2024 08:52
@phts
Copy link
Contributor Author

phts commented Feb 9, 2024

Updated my MR to support new version of go-librespot with mullti-user support (devgianlu/go-librespot#14, 0.13.0, however still waiting for this version to be released by author https://github.com/devgianlu/go-librespot/releases)

Rebased on #295

Commit with related feature: b6172fe

@phts phts force-pushed the spotify-multiuser-device branch from b6172fe to 6d9d7ed Compare February 9, 2024 11:18
@phts phts force-pushed the spotify-multiuser-device branch from 6d9d7ed to 80fedf8 Compare February 20, 2024 19:42
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.

2 participants