-
Notifications
You must be signed in to change notification settings - Fork 276
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
Torrentday: Switched back to the new rest api. #3727
Conversation
params = dict({'q': search_string}, **self.categories[mode]) | ||
|
||
if self.freeleech: | ||
params.update({'free': 'on'}) |
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.
I'm not sure if this still has any effect?
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.
afaik its to only list free leech torrents. @duramato confirm?
Last time i accessed TD It was available the information
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.
Yeah but is it available through the new api?
Codecov Report
@@ Coverage Diff @@
## develop #3727 +/- ##
===========================================
- Coverage 28.88% 28.78% -0.11%
===========================================
Files 277 279 +2
Lines 35005 35627 +622
===========================================
+ Hits 10112 10255 +143
- Misses 24893 25372 +479
Continue to review full report at Codecov.
|
reminder to myself. Remove the freeleech param. And filter using |
@p0psicles where is the API documentation? I couldn't find |
Don't think there is any |
Nice reading lol: https://www.torrentday.com/forums.php?t=3007437&page=1 |
@@ -138,6 +135,10 @@ def parse(self, data, mode): | |||
" minimum seeders: {0}. Seeders: {1}", title, seeders) | |||
continue | |||
|
|||
# Check if this is a freeleech torrent and if we've configured to only allow freeleech. | |||
if self.freeleech and row['download-multiplier'] != 0: | |||
continue |
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.
@p0psicles why not discard freeleech earlier before setting title
? title = re.sub(...
- line 120
With special tnx to: @kayl669