[artonge]

1. feat: Implements getSeenUsers to iterate over users
2. chore: Refactor callForSeenUsers to use getSeenUsers
3. feat: Limit ExpireTrash job to 30 minutes

Just wondering if we should use DateTime instead of int 🤷

Instead of adding a limit to callForSeensUsers it might be better to instead have a method that returns all (seen) users as an iterator. Then the call site can do it's own limiting.

Instead of adding a limit to callForSeensUsers it might be better to instead have a method that returns all (seen) users as an iterator. Then the call site can do it's own limiting.

Good idea and easy to implement :). Just pushed that version in the latest commit. Will remove the other change after review.

Note that this probably need to be backported

Looks good, you can also re-implement callForSeenUsers on top of getAllSeenUsers for some de-duplication