-
Notifications
You must be signed in to change notification settings - Fork 397
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
Multi Player Support #2164
base: future3/develop
Are you sure you want to change the base?
Multi Player Support #2164
Changes from 1 commit
c3549eb
5aaa2d1
059566d
7992662
6b89ddf
1890016
32d53af
00d1ea5
a19382b
98cb97d
d1536de
efb3590
01ff28d
83001f7
e365417
19ebc60
271c634
92135e9
6646f9a
6e0a3f8
5c1b84d
c74ead5
cbd59de
81143cb
93fc3a9
98da122
1c505b5
f3eeb6e
1b4b540
998ed06
ade2023
2b4e666
29cc50a
d4ddadc
e1c25a7
2d8ca15
a955afb
5aef09f
6b3b6fc
9928012
cbe4749
e12a53b
5bf3b40
d0f64f4
f7d5dff
a75b0d2
df89357
6e64466
5847dad
244a0cf
04fc9fc
2bf8fbc
de9cbb7
90b54c4
665a53f
8db761f
55d721a
5412b59
ece4c2b
d47087e
fbe43a0
c094839
ecf11dc
839f1ff
c94d5db
5e3bc6d
a118f70
d0832c9
4d3c46c
731c9a9
23a0f82
bd8af97
aa8443d
9d059b4
6ec73e2
c7a3eab
20d7284
6b54f47
ef92b71
1547a42
c8bc8c0
d0e851f
fd04413
aaa935a
b910210
86e734e
36bfd79
b8fd417
9006d6f
5784f8d
3b4f448
efbe969
e85b294
7a0d170
257d177
90cc33b
d43c6fe
e3ea953
e0faa44
3d7195d
0deda1e
9a506f0
af3b281
3e0c94a
8d65be3
bb24d8a
f34815d
d24e16a
190aaf7
2eccf39
633449e
3510085
3aedd9c
8cb99d1
afcafd1
8be8ade
e5ae214
3edfa67
c8f809b
fcf2ced
d38dc0c
d880d46
ba72a18
8169592
b4ff177
83692f1
42c15f4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -148,12 +148,12 @@ _option_spotify() { | |
print_c "---------------------- SPOTIFY ---------------------- | ||
|
||
Installs an additional player, so that you can | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am adding this comment without having reviewed the actually player logic. We should not have different players. If a user does not install Spotify, we should still install the same player which only uses MPD. With this logic, we would not install "an additional player" and instead, "just Spotify". But In understand that there is more to do than just providing credentials (e.g. Developer Account)
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't really get what you mean. The logic is: There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it's just different understand of wording. Ignore my comment for now. |
||
play music from spotify. | ||
play music from spotify. You will need to create | ||
an app in the Spotify developer dashboard to use | ||
Spotify properly | ||
|
||
Note: You need Spotify Premium to run this service! | ||
Note: Spotify Premium is needed to operate player! | ||
|
||
For more information see documentation: | ||
https://github.com/MiczFlor/RPi-Jukebox-RFID/blob/future3/develop/documentation/builders/spotify.md | ||
|
||
Do you want to enable the spotify player? [y/N]" | ||
read -r response | ||
|
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.
Sry for getting at it again, but i still think this should be ${HOME} as it is supplied as args and defaults to "root".
So right now if the user will provide different values it breaks.