Notifications
Implementation Stories, in priority order
I see Comment Notifications in Summary form: That is, all of the
new Comments in a single Space should get rolled up into a single visible Notification, whose header should be something like "14 new comments in Foo", and whose body shows as "3 comments on Bar; 5 comments on Floob", etc.
Also referred to by
Comment Management should be more sensible: This needs design, but is clearly a design bug. This business of having to manually turn on Comment Notifications is proving stupid -- On should probably be the default, at least for Members with higher Roles than Commentator. Also, the UI for managing it is idiotic: we need something more straightforward, probably hooked off the Notification Icon.
If you are logged out, and press the Notifications button, it hangs: That is, if I
was logged in in this window, so the UI thinks I'm logged in, but I've either timed out or logged out in another window, I can still press the button. It fails silently; it should give an appropriate "you aren't logged in" error, and a button to log in.
Notification Rendering no longer works appropriately: This design bug isn't user-visible -- yet -- but shows up as a horrible hack in the Client's NotificationsPage. CommentNotifier shows the bug clearly. The issue is that we're rendering the link to the Comment deep in the server, inside CommentNotifier, but we really should be doing so client-side.
Rethink defaults and UI for Comment Notifications: At the moment, you have to explicitly turn on Comment Notifications from the Advanced page if you want to get them. That's probably wrong: they should be on by default, I think. Regardless, we need better UI for them.
I see Comment Notifications in Summary form: That is, all of the
new Comments in a single Space should get rolled up into a single visible Notification, whose header should be something like "14 new comments in Foo", and whose body shows as "3 comments on Bar; 5 comments on Floob", etc.
Sending System Message scales poorly: When I send a System Message, I am sending it individually to every User. That's a nice easy way to start, but is going to deeply suck when we have 10 million users.
Closed Issues
I can see all of my Notifications: Initially, I click on my little bell icon, and get taken to a page showing all of the Notifications. Initially we don't worry about Summaries, since we just have the System Notifications.
Errors while sending out _notify(): Eric is trying to send mass emails from the Spirit Island Space. He's seeing erratic behavior, and I'm seeing errors in the Splunk logs.
I can see all of my Notifications: Initially, I click on my little bell icon, and get taken to a page showing all of the Notifications. Initially we don't worry about Summaries, since we just have the System Notifications.