[buddypress-trac] [BuddyPress Trac] #7135: Activity reply notifications should use the comment ID instead of the parent activity ID

[buddypress-trac]

#7135: Activity reply notifications should use the comment ID instead of the
parent activity ID
--------------------------+------------------
 Reporter:  r-a-y         |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  2.6
Component:  Activity      |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------

Comment (by r-a-y):

 Thanks for testing, @dcavins.

 Regarding
 [https://wordpress.slack.com/archives/buddypress/p1466615871000898 your
 comment on Slack]:

 > Related: if a member replies to a comment, should the comment author be
 notified?

 If a member posts a comment from the blog post page, a screen notification
 will not be added.

 `03.patch` solves this.  Screen notifications are now added if a comment
 is added from the blog post page and not just from the activity directory
 page.  See `bp_activity_add_notification_for_synced_blog_comment()` and
 the new `'bp_blogs_comment_sync_activity_comment'` hook.

 This piggybacks off of the activity reply notification format so "activity
 stream commenting on blog and forum posts" needs to be enabled.

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