-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Refactor code to use the same internal api for cli and interactive interface #79
Comments
Hi, @l-wegner, By that time enjoy this meme 👇 , hope you like it 😄 Use this action on your projects. Use jokes on issues instead. |
I'll start this tomorrow. |
yes, I also feel that. need to refactor before I add new features. we can discuss the design |
omg this would be amazing or even the ability to use socat to listen in to the dumped json so we can integrate into other apps like current song or station info |
did not get it! |
Is your feature request related to a problem? Please describe.
Currently there are function, which are available in cli and some are available in interactive interface. Adding new functionality is quite distributed.
Describe the solution you'd like
Having an internal API, which encapsulates the player, the recording, the favs, and the stations db, would allow cleaning up the main.py regarding the parsed arguments and allows for implementing a better interactive interface.
Further more extract the ui printing.
In the best case the
utilities.py
is almost empty afterwards.Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: