-
Notifications
You must be signed in to change notification settings - Fork 12
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
Application needs to be run several times to remove all comments/posts #35
Comments
Thanks for mentioning this. I noticed this behavior before, actually, but I thought maybe it was a bug that would have been fixed after I'd done a refactor. I wonder if it's something about the Reddit API itself. It seems to have a lot of odd behavior. I'll look into this. I had a workaround in place which was to try paginating through results again after the initial pagination had completed, and just keep continuing restarting pagination until it returned no results, but I had to remove that because it was causing an infinite loop when any posts could not be deleted. In the meantime you can just run it again after it completes. |
Workaround: looping bat file.
|
Had the same issue. Just using this with zsh. It's not fool proof but it will get the job done as a background task as I check up on it periodically.
|
Info update after some testing 👍 That said, over in #61, it appears that the GDPR export is a nice workaround to this. |
Thanks for looking into this more @RFBomb ! I had suspected this was the case. At one point I had some logic to restart the loop and keep going until the first API call returns no results. I removed that at some point, IIRC because sometimes posts can't be deleted for certain reasons, so those would cause infinite loops if you had enough of them. We could add it back and have the exit condition check that the last iteration through all results weren't all posts which could not be deleted or that the first page is empty. Still not great since it means theoretically you could have 34 pages of non-deletable posts and the 35th is deletable, but you can't get to it because the Reddit API is terrible. |
But yeah, seems like we should explain to users that GDPR is the only sure way to delete everything. |
More testing performed on this using the '?limit=100' property for the json request.
this jive's with reddit's 1k limit on their cache. https://www.reddit.com/r/redditdev/comments/2ffide/listing_old_comments/ |
https://github.com/j0be/PowerDeleteSuite EDIT: I just ran back to back tests, one using Shreddit and one using the PDS linked above. out of roughly 110 comments, PDS successfully edited about 30-40%, maybe less, after 3 runs. Shreddit (edit only mode) edited every single comment. |
I don't know if it's intended functionality, but the application stops after x amount of deletions, even if there are more posts left to delete.
I've run the command
As I said, it might be intended functionality, but if it isn't, I thought I'd let you know.
The text was updated successfully, but these errors were encountered: