-
Notifications
You must be signed in to change notification settings - Fork 190
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
Implement purgeUnread setting #1931
Conversation
cac1fd4
to
802d52b
Compare
This comment was marked as resolved.
This comment was marked as resolved.
Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
802d52b
to
da02122
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @Grotax
Co-authored-by: Sean Molenaar <SMillerDev@users.noreply.github.com> Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed - New administrator setting for deleting unread items automatically (#1931) Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed - New administrator setting for deleting unread items automatically (#1931) Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Changed - New administrator setting for deleting unread items automatically (#1931) Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
So to understand this correctly: before this merge the included autopurge would only delete already read articles? |
Yes, well it is still the default. You have to enable it. We are also not a 100% sure how many articles are kept but it will be around that number that you enter. |
This implements a new admin setting, allowing the admin to configure news to also delete unread items with every update.
Fixes #110