Skip to:
Content

BuddyPress.org

Opened 9 years ago

Closed 9 years ago

#5452 closed defect (bug) (fixed)

Order notifications by date

Reported by: mpa4hu's profile mpa4hu Owned by:
Milestone: 2.0 Priority: normal
Severity: normal Version:
Component: Toolbar & Notifications Keywords: ui-feedback reporter-feedback
Cc:

Description

For now notifications are ordered by component.

I had little hack which was including date with every notification and on client side I was resorting them with javascript.

That was wrong of course, mainly because I had core hacks to include date. And it was really hard to update that with every new update (glad I got rid of that)

But this is still a problem. Nor order it by date nor it should be visible (separated) that these notifications are coming from different components and thats why you got this in this order.


    

Change History (3)

#1 @boonebgorges
9 years ago

In 8153:

Add test for sort_order and order_by date in notifications component.

See #5452

#2 @boonebgorges
9 years ago

  • Keywords reporter-feedback added

Notifications can already be queried by date. Here's a unit test that shows how: https://buddypress.trac.wordpress.org/browser/trunk/tests/testcases/notifications/class-bp-notifications-notification.php#L119

Are you suggesting something in the UI? What would it look like?

#3 @DJPaul
9 years ago

  • Milestone changed from Awaiting Review to 2.0
  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.