Interface for transmission's labels feature

rndusr / stig

TUI and CLI for the BitTorrent client Transmission

GNU General Public License v3.0

554 stars 24 forks source link

Interface for transmission's labels feature #220

Open rsekman opened 2 years ago

rsekman commented 2 years ago

Since version 3.00 transmission now supports labels for torrents, e.g., music, movies, iso. stig should provide an interface to this feature, meaning

[x] torrent lists should have a column for labels
[x] torrent details should display labels
[x] it should be possible to filter torrents by labels
[x] there should be a command to manipulate labels
- [x] set a torrent's labels to a new list
- [x] add any number of labels
- [x] remove any number of labels
- [x] clear all labels
[x] add should take an argument to label torrents as they are added

rndusr commented 2 years ago

Thank you.

Looks great, apart from some style issues. Running flake8 should point those out.

rsekman commented 2 years ago

Thanks! I think I got all of them fixed.

Thank you also for pointing me to flake8. This will help me lint the rest of my contributions. Previously I was sort of guessing the style.

rndusr commented 1 year ago

I'm also using isort for keeping imports predictably ordered.

Also, I've decided I'm not going to put more work into stig's current code. It badly needs to be split into multiple projects, and the current design doesn't really allow for supporting more clients, at least not as easily as it should be. So, even though I generally welcome PRs, it's a waste of everyone's time at the moment.

But I'll pull your existing PRs and probably maybe make another release with them.