Skip to:
Content

BuddyPress.org

Opened 11 years ago

Closed 11 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
11 years ago

In 8153:

Add test for sort_order and order_by date in notifications component.

See #5452

#2 @boonebgorges
11 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
11 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.