feat: people infinite scroll

[michelheusschen]

Adds pagination for people to the server and infinite scroll to the people page on web for viewing more then 500 people.

• Order by person.createdAt to have results in the same order every time
• hasNextPage is optional to keep backwards compatibility with the mobile app
• Web will see some performance issues when loading a lot of people. I plan to address this in a future PR using virtual scroll.

[danieldietzler]

With the pagination not being optional this is a breaking change, correct? Besides, that, the code looks really good! You're on a rollllll

[danieldietzler]

I don't think you need the type annotation, do you?

[michelheusschen]

The conversion is necessary because implicit conversions are disabled. Pagination is optional with the same defaults as before, so there shouldn't be any breaking changes.

[danieldietzler]

Ah ok cool!

[jrasm91]

We only do because it is a query param so it would be a string otherwise.

[bo0tzz]

Order by person.createdAt to have results in the same order every time

This means the person page is no longer ordered by how many photos there are of a person, right?

[michelheusschen]

No, the current order is kept. If you have multiple unnamed people with the same photo count the order would previously be undefined which is a problem for pagination. Now it additionally sorts by person.createdAt to have a consistent sort order.

[mertalev]

The only thing with this is that it will scan and sort all the faces in the table to count them, then do that again for the next page. But that’s still much better than not being able to see the next page at all.