-
Notifications
You must be signed in to change notification settings - Fork 394
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
Added "Resume playlist" functionality and removed unused play func #30
Conversation
Giving the rc interface a "play" command while paused will actually give you the feedback to enter "pause" again. Additionally, the code went for the wrong key. Since pause does pause/unpause, there is no need for "play"
Added -F to grep to allow special characters in files.
Changed grep to sed to make files with special characters work better.
Giving the rc interface a "play" command while paused will actually give you the feedback to enter "pause" again. Additionally, the code went for the wrong key. Since pause does pause/unpause, there is no need for "play"
pinging @princemaxwell for review as per micz request :) I found one inconsistency that someone else might be able to look at: The web interface directly starts vlc with commands. This will circumvent this functionality. It would be better to start the .sh trigger script with a parameter, but I'm not too sure how to do that, so maybe micz might now. |
I fixed the website but I'm not sure why it says it conflicts. I'm sure micz will resolve that :) I basically changed the call from directly invoking vlc to starting the trigger script with the id. |
Content Successor to #24 as I wanted to clean up the commit. See that PR for discussion.