nextcloud / nextcloud/notifications
Group delete notifications
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 153
- Forks
- 67
- Avg merge
- 15h 3m
- Merged PRs (30d)
- 96
Description
At the moment one notification is sent for every deleted notification.
This is okay for normal NC notifications, but for Talk notifications, this might be improved.
Imagine the following case:
You have 100 unread messages/notifications in a chat.
Once you enter that chat, NC will send 100 delete push notifications to your mobile device (even if you have dismissed manually that notifications in your device).
Maybe we can group somehow that delete notifications? Like for example sending an array of notificationIds in just one delete notification.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing where delete notifications are produced and how their payloads are sent to clients. Define the grouped payload behavior for the Talk case, including notification IDs and compatibility expectations, then verify that a single grouped deletion replaces the current per-notification behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- api, backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100