photocrowd / django-cursor-pagination

Cursor-based pagination for Django
BSD 3-Clause "New" or "Revised" License
156 stars 27 forks source link

Fails to handle multliple ordering fields properly #6

Closed mjtamlyn closed 1 year ago

mjtamlyn commented 8 years ago

We need to ensure that only the last field is __lt or __gt, the previous ones should be __lte.

As an example, order on ('a', 'b'), page at 2.

Data:

 a | b 
 1 | 2
 1 | 3
 1 | 4
 2 | 1

"second" page will return only the last element.

danpalmer commented 6 years ago

Can this issue be closed?