Skip to content

Update Torrent model with missing API fields#34

Merged
George-Miao merged 2 commits intoGeorge-Miao:masterfrom
anriha:master
Aug 14, 2025
Merged

Update Torrent model with missing API fields#34
George-Miao merged 2 commits intoGeorge-Miao:masterfrom
anriha:master

Conversation

@anriha
Copy link
Contributor

@anriha anriha commented Aug 6, 2025

Thanks for the great work on this library. While using it, I found that the Torrent model was missing several fields that are available in recent versions of the qBittorrent API.

This PR addresses that by adding the following missing fields to the Torrent struct:

  • comment
  • download_path
  • has_metadata
  • inactive_seeding_time_limit
  • infohash_v1
  • infohash_v2
  • max_inactive_seeding_time
  • popularity
  • private
  • reannounce
  • root_path
  • trackers_count

It's worth noting that the official qBittorrent API documentation appears to lag behind the API itself. This update is based on the actual JSON response from qBittorrent v5.1.2 and should bring the model up-to-date with what users will encounter in the wild.

Let me know if you have any feedback or require any changes.

@George-Miao
Copy link
Owner

Would you please also update README with the comment in this PR? These undocumented fields may confuse users.

@anriha
Copy link
Contributor Author

anriha commented Aug 14, 2025

Thanks for the feedback. I've updated README to reflect this. Does this look good?

Copy link
Owner

@George-Miao George-Miao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for the PR!

@George-Miao George-Miao merged commit 6d01044 into George-Miao:master Aug 14, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants