Skip to content

Support second-level precision on searches against DateIndex#81

Merged
ramonski merged 17 commits intosenaite:2.xfrom
nnpvaan:fix-filter-timestamp
Mar 13, 2026
Merged

Support second-level precision on searches against DateIndex#81
ramonski merged 17 commits intosenaite:2.xfrom
nnpvaan:fix-filter-timestamp

Conversation

@nnpvaan
Copy link
Copy Markdown
Contributor

@nnpvaan nnpvaan commented Jan 12, 2026

Description of the issue/feature this PR addresses

This PR fixes timestamp filtering for API queries using created_since and modified_since fields. It ensures results are filtered with full second precision

Current behavior before PR

API queries with created_since or modified_since could return records with timestamps before the specified threshold

Desired behavior after PR is merged

API queries now filter records by the exact timestamp

--
I confirm I have tested the PR thoroughly and coded it according to PEP8
standards.

@nnpvaan
Copy link
Copy Markdown
Contributor Author

nnpvaan commented Feb 23, 2026

@xispa after checking the changes I noticed that the catalog date filter is missing in this commit:7051e5d...861ae75.
I re‑added the filter in this PR, could you help me review again?

@xispa xispa changed the title Enhance date filtering in catalog search to support second-level Support second-level precision on searches against DateIndex Mar 13, 2026
Copy link
Copy Markdown
Contributor

@ramonski ramonski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!
Works as expected for our use case.
It's a pity that the DateIndex has this 1 minute constraint.

@ramonski ramonski merged commit 3d5c593 into senaite:2.x Mar 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants