Skip to:
Content

bbPress.org

Opened 9 months ago

Last modified 7 months ago

#3341 new defect

Notifcations for new replies do not get marked as read

Reported by: konnektiv Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Extend - BuddyPress Keywords: has-patch
Cc: mattiejas

Description

When clicking on the notification for a new topic reply the notification is not marked as read an therefore does not disappear from the displayed notifications. This is due to the notifications are updated by item_id which is set as the topic_id, but are created with the reply_id as the notification item_id. The attached patch modifies the notification component action to include the topic_id and then marks all notifications as read with the same component action. This has the added benefit that if a user is notified of replies to different topics the user created these are shown as different notifications for each topic. Clicking on each notification opens the correct topic instead of just the first one as it is now.

Attachments (1)

fix-mark-notifications-as-read.patch (2.4 KB) - added by konnektiv 9 months ago.

Download all attachments as: .zip

Change History (3)

#1 @konnektiv
8 months ago

  • Keywords has-patch added

#2 @mattiejas
7 months ago

  • Cc mattiejas added
Note: See TracTickets for help on using tickets.