search

omaralvarez / deluge-autoremoveplus

Auto removing of deluge torrents
152 stars 61 forks source link

Maximum torrents add a checkbox to enable it #9

Closed ghost closed 9 years ago

ghost commented 9 years ago

It should be 0 not -1

omaralvarez commented 9 years ago

No, it should not, -1 means that the plugin is disabled. If a user would install the plugin and it would have 0 it would start deleting every torrent that meets the requirements, that is every single torrent with default settings. So this cannot be changed to 0 initially.

ghost commented 9 years ago

I understand that. We should add an hint somewhere because it's not really easy to understand that option... (Few users on the forum were stuck)

omaralvarez commented 9 years ago

Yes, you are right, maybe changing the label to (-1 to disable) or adding a check button would be better.

ghost commented 9 years ago

Exactly what I tought, a checkbox.