[buddypress-trac] [BuddyPress Trac] #8697: BP REST API Messages endpoint - total messages per thread

[buddypress-trac]

#8697: BP REST API Messages endpoint - total messages per thread
------------------------------------+--------------------------
 Reporter:  niftythree              |       Owner:  espellcaste
     Type:  enhancement             |      Status:  assigned
 Priority:  normal                  |   Milestone:  11.0.0
Component:  REST API                |     Version:
 Severity:  minor                   |  Resolution:
 Keywords:  dev-reviewed has-patch  |
------------------------------------+--------------------------
Changes (by prbot):

 * keywords:  needs-patch dev-reviewed => dev-reviewed has-patch


Comment:

 ''This ticket was mentioned in
 [https://github.com/buddypress/buddypress/pull/31 PR #31] on
 [https://github.com/buddypress/buddypress/ buddypress/buddypress] by
 [https://github.com/renatonascalves renatonascalves].''
 As titled.

 In this first part, I'm making sure data is present in a performant way.
 The second part is changing `get_messages` and `get_recipients` to stop
 getting all data at once.

 Currently, if a thread has 10k messages, `get_messages` will try to get
 all messages in a single SQL query. The same is true for `get_recipients`.
 This is a breaking change, so it bears more discussion.

 Trac ticket: https://buddypress.trac.wordpress.org/ticket/8697
 Trac ticket: https://buddypress.trac.wordpress.org/ticket/8696

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8697#comment:4>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac