reddsaver

reddsaver copied to clipboard

Using the -U flag with reddsaver unsaves whatever it skips, not just what it successfully downloads.

I was hoping to use this function to free up access to saved posts after the 1000 post limit. However it caused some issues.

In my case (running Windows 10 with the human readable flag):

• It unsaved posts that had a WARN (as in this issue). This is fine because I can go back through the log and find the posts on reddit again manually.

• It unsaved self-posts/text posts. This is worse as it doesn't log these and I have lost hundreds of self-posts I had saved.

It is my fault for making assumptions about the functionality, but I think this should be mentioned somewhere so others don't lose their saves.

Thanks @troutmouth for bringing this up. Sorry that you ended up losing your saves.

By self-posts you mean posts that you created and saved?

Currently, no distinction is made on who created the post. Do you think an option to limit unsaving only the non-self-posts would be useful? Also, would the unsaves to only successfully completed ones be useful?

I don’t use the unsave option myself, so knowing how people expect it to work would be valuable.

I am referring to text posts made in a subreddit that are not link-only posts.

Here is an example of one. You will see self.[name of the subreddit] next to the post title. In the example link, it is "(self.photography)".

I have also found another issue with this just now testing it (and made my own problem worse lol): I used the --subreddits flag to try to limit reddsaver to one sub. I had saved a post from a random sub I don't have any other saves from to try and test if it would keep it or not. Not only did reddsave unsave ALL self.posts outside of that sub from the next 1000 saves I had left, it unsaved all image/video posts too even though it made no attempt to download them.

Also, would the unsaves to only successfully completed ones be useful?

This would be perfect.

Oh that’s bad! Sorry about that - Might be a good idea to try it on a different account or hold off on using unsave altogether for the moment 😅

Thanks for reporting these, this is very helpful. I will try to revamp the unsave implementation.