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

Add a user setting to automatically mark fetched posts (list_posts) as read. #5144

Open
5 tasks done
dessalines opened this issue Oct 28, 2024 · 0 comments · May be fixed by #5160
Open
5 tasks done

Add a user setting to automatically mark fetched posts (list_posts) as read. #5144

dessalines opened this issue Oct 28, 2024 · 0 comments · May be fixed by #5160
Labels
enhancement New feature or request
Milestone

Comments

@dessalines
Copy link
Member

Requirements

  • Is this a feature request? For questions or discussions use https://lemmy.ml/c/lemmy_support
  • Did you check to see if this issue already exists?
  • Is this only a feature request? Do not put multiple feature requests in one issue.
  • Is this a backend issue? Use the lemmy-ui repo for UI / frontend issues.
  • Do you agree to follow the rules in our Code of Conduct?

Is your proposal related to a problem?

Users want the ability automatically mark posts in their feed as read, without having to manually click into the post, or click a mark read button.

Currently, a few apps are doing this by collecting up ids, then when its scrolled to the bottom, send a list of post ids to the mark_read function. But #5043 takes this away, in exchange for returning the full post result.

At the same time, collecting up ids isn't the cleanest solution, and most apps probably don't want to code this. We also don't want to spam a lot of individual, or collected requests.

Describe the solution you'd like.

Add a user-setting to automatically mark fetched posts (the posts received back from list_posts, as read.

Pros

  • Easy to implement on the front and back-end, requires no more than adding the setting.
  • No more spamming mark read.

Cons

  • A large fetch size might mean that some posts that weren't seen (beyond the scrolling / visible region), will get marked as read, even though they weren't seen.

Describe alternatives you've considered.

NA

Additional context

#5043 (comment)

@dessalines dessalines added the enhancement New feature or request label Oct 28, 2024
@dessalines dessalines changed the title Add user setting to automatically mark fetched posts (list_posts) results as read. Add a user setting to automatically mark fetched posts (list_posts) as read. Oct 28, 2024
@dessalines dessalines added this to the 0.20.0 milestone Nov 1, 2024
dessalines added a commit that referenced this issue Nov 2, 2024
- Rather than apps collecting up viewed posts ids, and sending many
  mark as read requests, users can now turn this setting on, and any
  results from /post/list will be auto-marked as read.
- Fixes #5144
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

Successfully merging a pull request may close this issue.

1 participant