Notification grouping now on master branch...
-
@julian said:
I initially wanted to group them as they were created, although it is difficult given that each individual notification can be shared with other users.
It gets complicated in situations like:
- Notification: User replied to topic
- Notification pushed to 4 users
- 2 of those users read that notification
- A different user replied to that topic
Now, what happens here? Do we modify the existing notification object? If we do, then the two users who read the notification already will receive a notification with incorrect data (that is, it will say two users replied to a topic, when it should say 1).
In hindsight, it might actually be better to keep notifications granular, and have each individual client handle duplication separately...
Do not send notification to the users who didnt read these notifications. In @exodo we modified the notifications and it doesn't send notification to the user who hasn't read last notification about something. For example: if someone favourited your post, you wont receive more notifications about more favs for that post until you read that post.
-
@alberto__segura that works just fine
no need to change what works really.I wanted to have notifications that are always correct. If two users reply, the notification recipient would only get one notif. That's okay, but could be better. These grouped notifications do the trick.
-
@julian said:
@alberto__segura that works just fine
no need to change what works really.I wanted to have notifications that are always correct. If two users reply, the notification recipient would only get one notif. That's okay, but could be better. These grouped notifications do the trick.
yes, grouped notifications are the best solution for the web notifications, but we thought about the plugin and email notifications. If you use the Telegram Bot plugin or the pushbullet plugin, you will receive a lot of notifications from the same topic or post, we doesn't like it, so we tried to fix it. I think both solutions could work together pretty well
-
@alberto__segura offhand, I do not know if those services support changing a notification if it had already gone out.
That'd be best...
-
@julian said:
@alberto__segura offhand, I do not know if those services support changing a notification if it had already gone out.
That'd be best...
I used the telegram plugin and it's true that I was receiving too many notifications.
Perhaps if the forum will send 2 variables it will be easier for the plugin to cull the notifications to avoid sending too many. For example: one variable might say the type of notification (if it's a mention, a message favourited, new answer, etc. ) and another variable might say if there is another mention, message favourited or new answer that haven't been read in that post. (@alberto__segura may that works?)
After all, from what I've seen (as a user) is that little things is what makes people think that nodebb has hundreds of differences with other platforms.
-
near-short-medium-long-term
Well at least he's honest. I just say it's coming soon
-
@JaredBusch Doesn't look like it.
-
@JaredBusch @julian I believe notifications route and drop down only show notifications for the past 7 days. If you don't have any in the past 7 days then it will be empty.
fat fingers caused me to mention the wrong fellow.
