Skip to:
Content

BuddyPress.org

Opened 3 years ago

Closed 3 years ago

#8547 closed defect (bug) (fixed)

BP Nouveau: restore the Notice notification feature

Reported by: imath's profile imath Owned by: imath's profile imath
Milestone: 10.0.0 Priority: normal
Severity: normal Version:
Component: Templates Keywords: has-patch
Cc:

Description

I just noticed this feature wasn't working anymore. I believe the Notifications component got some improvements with the introduction of the bp_notifications_get_grouped_notifications_for_user() function.

So the attached patch suggest to restore it. This is something I added to this template pack because Notices are displayed into the User's front-end profile and might not be seen by some users.

Attachments (1)

8547.patch (4.3 KB) - added by imath 3 years ago.

Download all attachments as: .zip

Change History (3)

@imath
3 years ago

#1 @imath
3 years ago

Here's a screenshot of the result of the patch when a notice is published widely.

https://cldup.com/8TL3qFemgH.png

#2 @imath
3 years ago

  • Owner set to imath
  • Resolution set to fixed
  • Status changed from new to closed

In 13036:

BP Nouveau: include a notification to the active Site Wide Notice

When a Site Wide notice is active the BP Nouveau template pack is prepending a notification to the logged in user notifications list.

FYI: This feature was "missing in action" since the introduction of the bp_notifications_get_grouped_notifications_for_user() and its use inside the bp_notifications_get_notifications_for_user() function in replacement of bp_notifications_get_all_notifications_for_user().

Fixes #8547

Note: See TracTickets for help on using tickets.