flood icon indicating copy to clipboard operation
flood copied to clipboard

feat: Add basic Category Support for qBittorrent

Open inochisa opened this issue 4 months ago • 5 comments

Description

Introduce basic category support for qBittorrent.

These PR still have some gaps:

  1. The category can not be created directly in flood.
  2. The savepath of category is not applied when setting category.
  3. The category setting is not applied for feed torrents.

Related Issue

#686

Screenshots

Types of changes

  • [ ] Breaking change (changes that break backward compatibility of public API or CLI - semver MAJOR)
  • [x] New feature (non-breaking change which adds functionality - semver MINOR)
  • [ ] Bug fix (non-breaking change which fixes an issue - semver PATCH)

inochisa avatar Aug 07 '25 07:08 inochisa

I'll merge this PR with squash, please do not force-push

trim21 avatar Aug 07 '25 08:08 trim21

I'll merge this PR with squash, please do not force-push

Sorry, I will do not do this again. Thanks for your reminder.

inochisa avatar Aug 07 '25 08:08 inochisa

The savepath of category is not applied when setting category

this involve the torrent auto manage feature in the qb and it's a little bit complicated, for now I think make category visible/filterable but not changeable in floold maybe a good idea. we can add these things later.

trim21 avatar Oct 21 '25 09:10 trim21

this involve the torrent auto manage feature in the qb and it's a little bit complicated, for now I think make category visible/filterable but not changeable in floold maybe a good idea. we can add these things later.

Good to know, I will update the code and add minimum features. Thanks

inochisa avatar Nov 16 '25 00:11 inochisa

Codecov Report

:x: Patch coverage is 58.95522% with 55 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 73.06%. Comparing base (15f7d23) to head (6eb18ca). :warning: Report is 41 commits behind head on master.

Files with missing lines Patch % Lines
...erver/services/qBittorrent/clientRequestManager.ts 11.76% 15 Missing :warning:
server/routes/api/torrents.ts 60.00% 14 Missing :warning:
server/services/taxonomyService.ts 65.71% 12 Missing :warning:
server/services/Deluge/clientGatewayService.ts 37.50% 5 Missing :warning:
...erver/services/qBittorrent/clientGatewayService.ts 69.23% 4 Missing :warning:
...rver/services/Transmission/clientGatewayService.ts 75.00% 2 Missing :warning:
server/services/rTorrent/clientGatewayService.ts 75.00% 2 Missing :warning:
server/services/feedService.ts 0.00% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #902      +/-   ##
==========================================
- Coverage   73.08%   73.06%   -0.02%     
==========================================
  Files          66       66              
  Lines       11533    11674     +141     
  Branches      987      990       +3     
==========================================
+ Hits         8429     8530     +101     
- Misses       3088     3144      +56     
+ Partials       16        0      -16     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Nov 16 '25 06:11 codecov[bot]