Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(dav): introduce paginate with custom headers #48662

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

feat(dav): introduce paginate with custom headers #48662

wants to merge 3 commits into from
+334 −1

Conversation

Altahrim
Copy link
Collaborator

@Altahrim Altahrim commented Oct 11, 2024 •
edited
Loading

Summary

Checklist

@Altahrim Altahrim added this to the Nextcloud 31 milestone Oct 11, 2024
@Altahrim Altahrim self-assigned this Oct 11, 2024
@icewind1991
Copy link
Member

pushed a minor change to better type return arrays

@icewind1991
fix: switch paginate cache query to use datetime instead of timestamp…
23b58c6 
… for 'insert_time'

this is needed to make it work with postgresql

Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
Copy link
Member

Changed the queries to use DateTimes instead of timestamps for the insert_time to make them work with postgresql

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@icewind1991 icewind1991 Awaiting requested review from icewind1991

At least 2 approving reviews are required to merge this pull request.

Assignees

@Altahrim Altahrim

Labels
2. developing Work in progress feature: dav
Projects
📁 Files team
Status: 🏗️ In progress
Milestone
Nextcloud 31
Development

Successfully merging this pull request may close these issues.
2 participants
@Altahrim @icewind1991