Skip to:
Content

BuddyPress.org

Changes between Initial Version and Version 1 of Ticket #5251, comment 7


Ignore:
Timestamp:
11/20/2013 11:04:48 PM (11 years ago)
Author:
johnjamesjacoby
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #5251, comment 7

    initial v1  
    33I believe this worked identically in 1.8, can you confirm?
    44
    5 If so, this is a consequence of how BuddyPress core notifications are cleared -- each component handles this in their own unique way. In the future, Notifications should have a universal `bp_notifications_clear` action, that fires before template output, to handle this for us, maybe even without redirect.
     5If so, this is a consequence of how BuddyPress core notifications are cleared -- each component handles this in their own unique way. In the future, Notifications should have a universal `bp_notifications_clear` action, that fires before template output, to handle this for us, maybe even without redirect (I.E. with ajax.)
    66
    77Related, we should nonce all of these notifications links in 2.0. Marking and deleting notifications isn't a huge deal, but it's currently possible to trick someone into hiding notifications. It's not a regression, so I left it untouched for 1.9 until the dust settles.