Closed Dominyk4s closed 1 year ago
hey @Dominyk4s, thanks for opening this PR! I'll review it today/tomorrow
@Dominyk4s there was a similar PR that addressed a couple of the changes in the COLO switchover, can you review the conflicts?
@Dominyk4s there was a similar PR that addressed a couple of the changes in the COLO switchover, can you review the conflicts?
@mattpodolak, I reviewed another PR, which seems to solve the same thing, just with a different approach (changes the defaults there and I'm adding everything as variables here). And I see you moved forward with even more updates, so we can close this PR 😊
Just a quick temporary fix TL;DR: Added two parameters: _sortvar and _checktotals to API calls; add variables
sort_var='order', check_totals=False
while callingsearch_submissions
or any other function for it to work with the latest data in PushshiftLonger version:
After moving to COLO (related Reddit post is here), The current wrapper stopped working. Known differences: