[buddypress-trac] [BuddyPress Trac] #6057: Comment notification NOT notified

[buddypress-trac]

#6057: Comment notification NOT notified
---------------------------------------+-----------------------
 Reporter:  sebacar                    |       Owner:  imath
     Type:  enhancement                |      Status:  assigned
 Priority:  normal                     |   Milestone:  2.5
Component:  Component - Notifications  |     Version:  2.0.3
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch                  |
---------------------------------------+-----------------------
Changes (by imath):

 * keywords:  needs-refresh => has-patch


Comment:

 6057.02.patch is doing the following:

 1. manage screen notifications for activity comments and activity comment
 replies
 2. if there's only one notification of the above types, the WP Admin Bar
 bubble links to the single activity. Clicking on the link will mark the
 notification as read.
 3. if there's more than one, we use the user's notifications screen and an
 extra parameter to filter the notifications according to the type so that
 only these notifications are displayed. The notification won't be marked
 as read at this step because the content of the notified comment or reply
 is not displayed from this screen. Clicking on the entry link will mark
 the notification as read as it's the activity permalink.
 4. add new unit tests to prevent regressions in
 `bp_activity_format_notifications()` and in
 `bp_activity_new_comment_notification()` and to include some new specific
 tests to activity comment notifications.

 If we agree on the logic described in points 2 & 3, i think it's good to
 go :)

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6057#comment:21>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac