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 /next command #518

Merged
merged 3 commits into from
Apr 9, 2022
Merged

Add /next command #518

merged 3 commits into from
Apr 9, 2022

Conversation

rtix
Copy link
Contributor

@rtix rtix commented Mar 22, 2022

/next, /qn command to load next file in playlist. I find it useful for me, hence the proposal

@Et0h
Copy link
Contributor

Et0h commented Apr 6, 2022

Nice work. I've tested it and it operates correctly. It's a simple change to do the code so shouldn't add any bugs, and I can see why it would be useful for those controlling Syncplay through command line and mpv.

Unless anyone objects I'd say it's good to accept, with the remaining tasks being to:

@rtix
Copy link
Contributor Author

rtix commented Apr 9, 2022

@Et0h I've added stubs, but I'm not sure how this command should look like in docs and how do I contribute to website?

@Et0h
Copy link
Contributor

Et0h commented Apr 9, 2022

@Et0h I've added stubs, but I'm not sure how this command should look like in docs and how do I contribute to website?

Only I can update the website. I'll do the docs as well.

@Et0h Et0h merged commit b7b80df into Syncplay:master Apr 9, 2022
@Et0h
Copy link
Contributor

Et0h commented Apr 9, 2022

Just checked and at present manual is only for command line options, not /commands. As such I've just updated the website.

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