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

group notifications by thread only, not by thread/user #51

Open
Fusselwurm opened this issue Sep 28, 2017 · 1 comment
Open

group notifications by thread only, not by thread/user #51

Fusselwurm opened this issue Sep 28, 2017 · 1 comment

Comments

@Fusselwurm
Copy link
Member

One notification by thread should be enough, we dont need 20 notifications for different users, when the notification links to a place where you see all the changes in one place anyway.

-> Group by thread, concat user names.

@Fusselwurm
Copy link
Member Author

🤦‍♀️

implementation right now:

  • I create one notification for "user a has done x"
  • then I push this notification to everyone who is subscribed to the thread in any way

what we'd need here:

  • get all people who are subscribed to the thread
  • for each, get the respective "users a,b,c did x" notification and update it with another user

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

1 participant