N4S4 / synology-api

A Python wrapper around Synology API
MIT License
360 stars 137 forks source link

Command Line Interface Support #129

Closed fortysix2ahead closed 1 year ago

fortysix2ahead commented 1 year ago

Is your feature request related to a problem? Please describe. Already started to implement CLI support (focused on Syno Photos) to be able to remote control a syno machine. First results are available here, on feature branch: https://github.com/fortysix2ahead/synology-api/tree/feature/cli

Describe the solution you'd like In case there is interest, a new branch in the main repository (feature/cli) would be nice so I might push the fork onto this branch. The CLI package is parallel to the api package, but introduces new dependencies that might be unwanted, that's why a separate branch would be good option (at least for now)

Describe alternatives you've considered n/a

Additional context n/a

N4S4 commented 1 year ago

Indeed is a good idea to keep it in another branch, honestly I was thinking to do the same but great that you put your effort in it and thank you! I will be glad to keep the cli branch!

fortysix2ahead commented 1 year ago

Great. You need to create the dedicated branch in this repo before I can do a Pr onto that branch. I'll update this ticket when I'm ready, need to write some documentation first ...

N4S4 commented 1 year ago

Will do! Thanks for the work!

N4S4 commented 1 year ago

created a new branch at https://github.com/N4S4/synology-api/tree/synology-api-cli

fortysix2ahead commented 1 year ago

Did a PR onto the new branch, including documentation in README.cli.md. I think we can close this issue.