-
Notifications
You must be signed in to change notification settings - Fork 33
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
[Feature] Add option to set 'obsolete'-tag to private torrents that are not needed but kept anyway. #194
base: dev
Are you sure you want to change the base?
Conversation
hi @Dark3clipse, thanks for your contribution. The behavior is not entirely clear to me, here are a few thoughts.
|
Thank you for the feedback, I will rework the code. |
It is not clear to me what you mean with: We have no knowledge of what other tools are managing/using qbit (such as qbitmanage). Also, I think decluttarr should not have to know. You just set the feature to True if you want to use it imo. qbit itself is not 'using' the tags. I do check the existence of the tag in qbittorrent (see loadScripts.py) and create it if the feature is enabled and the tag string is not empty. |
Except for your 3rd point, the code should be reworked now. Could you have another look to see if your concerns have been addressed? Thank you :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey!
Almost there I think.
Please can you have a look at my comments.
Also, I saw the unit tests failed in the github actions, pls can you check?
I'll need some time due to Christmas with in laws, I will come back and address the remaining work in a couple of days if that's okay. Again, thank you a lot for the feedback. Let's make this feature solid :) Wish you a nice holiday season as well. |
I've resolved the issues and applied your requested changes. Let me know what you think. Apologies for the delay, it has been a busy time for me. Also I've made sure the tests are passing now. |
Description
This feature introduces an option to tag torrents in qBittorrent that are determined to no longer be needed by the *arrs. This allows third-party tools to remove these torrents at a later stage when seeding requirements are met.
Use case
You have private trackers and want to make sure that you meet the seeding requirements before removing these torrents. Sometimes, torrents are no longer needed by the *arrs. You would like to remove these torrents after the seeding requirements are met. A tool like qbit_manage can do this for you.
However, such tools would need to know which torrents are no longer needed (obsolete) by the *arrs. By making decluttarr tag these torrents, third party tools such as qbit_manage can be used to cleanup these torrents after seeding requirements are met.
Changes