Skip to content
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

Feature - Remove posts from a channel #17

Open
jackjamieson2 opened this issue Aug 22, 2018 · 4 comments
Open

Feature - Remove posts from a channel #17

jackjamieson2 opened this issue Aug 22, 2018 · 4 comments
Labels
enhancement New feature or request

Comments

@jackjamieson2
Copy link
Owner

Thinking the best implementation would be to set the post contents to null. This way the post will still be saved so will not be re-fetched by mistake.

@jackjamieson2 jackjamieson2 added the enhancement New feature or request label Aug 22, 2018
@bekopharm
Copy link

Probably related: I got a stuck post that reappears all the time due to a date mistake on publish. Deleting doesn't help.

Screenshot_2020-01-18-18-19-38

@jackjamieson2
Copy link
Owner Author

Thanks @Bekoli, definitely related. The way it's set up now, Yarns will continually re-fetch that post since the date is new.

  1. Need to implement a way to delete posts so they won't be re-fetched.
  2. Thinking about how to handle posts with future dates... Probably best to leave as-is, even though it messes up the feed order. Alternately, if the date is in the future, Yarns could set the feed item's published field to the time of fetching. I don't like the idea of modifying this information (essentially falsifying it) but it might make sense in this case because the publication date is malformed to begin with. Third option is to refuse to fetch posts that are newer than the present date. Compromises abound

@bekopharm
Copy link

I see. That's a tricky one indeed. The third option is probably a bad idea. Half of the time there is no date that can be parsed anyway.

@jackjamieson2
Copy link
Owner Author

Agreed. Anyway, this is good motivation to at least (finally) implement deleting timeline items. I'll move that to the top of my list, and hopefully get to it soon. Been a bit difficult to be quick with updates recently

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants