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

As Chris I would like to see my old notification so that I am not getting confused #2140

Open
christianbrb opened this issue Sep 17, 2020 · 1 comment

Comments

@christianbrb
Copy link
Contributor

User Story

As Chris I would like to see my old notification so that I am not getting confused

@weilbith Is this correct like this? Please correct if this is not.

Acceptance criteria

Tasks

@weilbith
Copy link
Contributor

weilbith commented Sep 17, 2020

Sounds good.

This feature will then lead to the issue to make a decision how the notifications should be stored. Because the list of notifications is meant to grow statically over time, this will lead to storage space and thereby performance issues.
Therefore I suggest to tackle the assumption of statically growing and establish a clean up logic. Such mechanism should simply tidy up old notifications, so the list does not get to big at any time. It could be based on time (like purge notifications older than 30 days) or absolute number (like keep the last 1000 notifications). I would recommend to take the absolute number approach. Despite it is incredibly easy to implement, it guarantees that there are never too many notifications stored (for the theoretically case that the users causes millions of notifications within the defined time range). Also does this approach makes sure there are always some notifications shown. Never mind if he wasn't active for a long time or simple does not trigger that many notifications.

@taleldayekh what do you think?

@christianbrb christianbrb added this to the Product Backlog milestone Oct 6, 2020
@taleldayekh taleldayekh removed this from the Product Backlog milestone Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants