-
Notifications
You must be signed in to change notification settings - Fork 9
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
Download New Episodes for Favorite Shows #14
Comments
What would you think of just adding a boolean value to each podcast called |
That's a better idea. It also prevents the breaking up of information about an individual podcast into multiple sections. It keeps all the data about the show in one place. Much better. Do you have any preferences about module structure? Is there a published pattern you're following that I can conform to if I take a pass at this? |
Honestly, it's not too rigid. I'm trying to follow the cobra guidelines for the different commands. I'd also still like to have the program available as a library and the cli is just an interface on it. Other than that, just the general go stuff (gofmt), introduce as little dependencies as possible, ... Nothing too crazy :) In any case: thanks for helping and enjoy ;) |
Thanks for starting the project! Let me know if I hit out of scope features. You've done great stuff here and I think it could easily become a really powerful library, but I want to be respectful of your goals. I'm already tinkering with integrating it into my own podcast flow. ;) |
Is your feature request related to a problem? Please describe.
I'd like to have the ability to have the tool download the latest episode of all shows in a list of favorite episodes in the config whenever a
sync
is run.Describe the solution you'd like
A new property in the config YAML will need to be added. I recommend an array of IDs:
Additional context
This will probably require some significant refactoring of the
download.go
file. I'm happy to work on it if you're open to that.The text was updated successfully, but these errors were encountered: