-
Notifications
You must be signed in to change notification settings - Fork 2
Pagination of Messages #447
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
... and 355 files with indirect coverage changes 🚀 New features to boost your workflow:
|
@Magnus-Kuhn why is this a breaking change? |
The total number of messages is returned as well in the use case as |
IMO no real reason. We can keep the connector api the same and we can update the App accordingly. |
Readiness checklist
Description
A first bare-bones implementation using the existing
docdb-database-abstractions
pagination without a default or restrictions for the requested pagination.A bug is that
count(query: {})
on lokijs always returns 0, we run into this because the querytranslator translates an undefined query to {}.