#5724 closed enhancement (duplicate)
bp_has_notifications to support all notifications
Reported by: | SGr33n | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | Toolbar & Notifications | Keywords: | |
Cc: |
Description
Hi,
Is it possible to modify bp_has_notifications to return all notifications? is_new argument is a boolean, so looks like it's only possible to request read or unread only.
Would be useful also to have an "All" link.
Thanks :)
Change History (2)
Note: See
TracTickets for help on using
tickets.
I'd say this is a duplicate of #5558.
Give the patch in that ticket a test and post a reply on that thread.
Update: I misread your ticket, SGr33n.
My patch does not address querying all unread and read notification together in
bp_has_notifications()
.It's supposed to work when passing
'is_new'
as the string'both'
. (See #5315.) But, you're right about the boolean restriction.Since my patch in #5558 covers more use-cases and I don't want to refresh that patch, I'll try to add these fixes in #5558.