Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Notification Expiry #337

Closed
julianlam opened this issue Sep 24, 2013 · 2 comments
Closed

Notification Expiry #337

julianlam opened this issue Sep 24, 2013 · 2 comments
Assignees
@julianlam
Labels
bug enhancement
Milestone
0.1.0

Comments

@julianlam
Copy link
Member

Currently, notifications don't delete themselves, which cause littering of the Redis database.

Utilise node-cron so that:

  1. If a notification has been read by all users and is over 7 days old, delete it (key notifications:{nid}
  2. If a user has read a notification and that notification is over 7 days old, remove it from the read set.
@ghost ghost assigned julianlam Sep 24, 2013
@julianlam
Copy link
Member Author

  • New notifications set
  • Prune expired notifications

@julianlam
Copy link
Member Author

Closed via 8b5cf0c

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@julianlam julianlam

Labels
bug enhancement
Projects
None yet
Milestone
0.1.0
Development

No branches or pull requests
1 participant
@julianlam