-
Notifications
You must be signed in to change notification settings - Fork 465
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
Add 1337x search plugin #43
Conversation
8c3e579
to
28438d8
Compare
Does this change only affects the newer releases or old ones too? AFAIK, old plugin engine versions don't support magnet links. |
@v1k45 affects all qbittorrent versions, users with old versions too. It should work in all versions. |
@Chocobo1 could you do the pep8 fixes for me? |
OK, but it will be later, short on free time right now... |
28438d8
to
b980f61
Compare
b980f61
to
bfcd9f7
Compare
I did, all PRs are ready to review. |
@@ -1,5 +1,6 @@ | |||
btdb: 1.04 | |||
eztv: 1.01 | |||
leetx: 2.00 |
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.
why the name is different? can't you use 1337x
?
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.
The file name must be the same as the class name and the class name can't start with numbers.
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.
ok, no more question from me, proceed as you see fit.
Based on @v1k45 source code https://github.com/v1k45/1337x-qBittorrent-search-plugin
Changes: