Download by series, download by author #184
Conversation
|
It looks quite good so far. One point should be considered again. What if someone is looking for a title and author. Then only audio books should be displayed where the title and author match. Currently, he would search by title or author and add the results! |
|
That is true; I assumed that was the way you wanted to do it since that's how it's set up with ASNs and titles, but if you want I can probably figure out how to do an intersection instead |
|
Maybe the |
| help="title of the audiobook (partial search)" | ||
| ) | ||
| @click.option( | ||
| "--author", "-a", |
There was a problem hiding this comment.
The -a option is already used by --asin.
There was a problem hiding this comment.
Oops =] I thought I'd looked for that.
All things considered, I think the more natural assumption would be to have the different lists merged before checking -- I just haven't had time to come back to this since then and I'm not sure how soon I can
There was a problem hiding this comment.
I think I'll have time this weekend to rework some things and then merge your request.
Implements #183
My python is more than a bit rusty; I tried to follow existing patterns, but let me know if there is anything you need me to fix.