Security Alert: 26 BuddyDev accounts compromized in brute force attack.

BuddyDev

Search

Activity Comment Notifier redundant notifications?

Tagged: 

  • Participant
    Level: Initiated
    Posts: 10
    Jamie Halmick on June 3, 2016 at 7:59 pm #4095

    Our site members are complaining that they get a separate notification for EVERY reply to a comment thread they are included in. I notice that in the source code of the plugin there is a line that would remove previous notifications for an activity but that it is commented out. Is there an issue with uncommenting that? Would it make it so there is only a single notification for each thread?

    See /core/actions.php line 24: //ac_notifier_remove_notification( $activity, true );

    Thanks!

  • Keymaster
    (BuddyDev Team)
    Posts: 24706
    Brajesh Singh on June 4, 2016 at 12:48 am #4102

    Hi Jamie,
    Welcome to BuddyDev.
    I am sorry but I don’t understand why will you use Activity notifier plugin if your users don’t want to be notified of the comment.

    That is the core idea of this plugin.

    can you please help me understand a little better how you want to use this plugin?

    Thank you
    Brajesh

  • Participant
    Level: Initiated
    Posts: 10
    Jamie Halmick on June 4, 2016 at 1:48 am #4105

    They do want to be notified of comments. However, when there have been multiple comment replies on the same post since they were last on the site they see multiple separate notifications that show up in the list. All of those notifications link to the same parent activity post. It would be better it seems if there was only one notification.

  • Keymaster
    (BuddyDev Team)
    Posts: 24706
    Brajesh Singh on June 6, 2016 at 3:30 pm #4122

    Hi Jamie,
    Thank you. I guess I understand it better now.
    The new notifications screen lists the redundant notifications while the dropdown does not. I am updating it in next 24 hours to account for the same.

    Thank you
    Brajesh

  • Participant
    Level: Initiated
    Posts: 10
    Jamie Halmick on June 8, 2016 at 5:13 pm #4154

    Perhaps that would be a solution. Maybe just mark the old redundant notifications as read so they don’t show up in the dropdown?

  • Keymaster
    (BuddyDev Team)
    Posts: 24706
    Brajesh Singh on June 8, 2016 at 9:22 pm #4157

    Hi Jamie,
    I am sorry for the delayed reply. I have looked at it in details and the update of the plugin will take me some time. There are issues that I have noticed. For example, some of the notifications were deleted automatically. I am planning to rewrite the whole experience to use new BuddyPresss advances(like notification meta etc) in next 1-2 weeks. For the time being, Please download the updated version from here for a temporary fix(Instead of duplicate notifications, It uses individual user in the message for notifications)

    https://github.com/sbrajesh/bp-activity-comment-notifier

    Hope that helps.

  • Participant
    Level: Initiated
    Posts: 10
    Jamie Halmick on June 8, 2016 at 9:41 pm #4158

    Thanks for the update! I’ll test this version out for now.

  • Participant
    Level: Initiated
    Posts: 10
    Jamie Halmick on June 20, 2016 at 5:05 pm #4313

    Any progress on the new version of this plugin? Thank you!

  • Keymaster
    (BuddyDev Team)
    Posts: 24706
    Brajesh Singh on June 20, 2016 at 5:10 pm #4314

    Hi Jamie,
    Were you able to test the git version? I will start working on it next week after publishing our upcoming theme. I am sorry for the delay.

  • Participant
    Level: Initiated
    Posts: 10
    Jamie Halmick on June 20, 2016 at 5:14 pm #4315

    I tested it. The issue with the git version is that the notification count on the bell shows the total notifications but then the dropdown only shows one notification. It seems it would be too confusing to our users. We have a few users that are really concerned about their notifications so anything being off is problematic. 😉

You must be logged in to reply to this topic.

This topic is: not resolved