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.

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

Use Implementation of ge/le for gt/lt in Date Search Params #410

Closed
alexanderkiel opened this issue Jun 17, 2021 · 0 comments
Closed

Use Implementation of ge/le for gt/lt in Date Search Params #410

alexanderkiel opened this issue Jun 17, 2021 · 0 comments
Assignees
Labels
enhancement New feature or request module:db
Milestone

Comments

@alexanderkiel
Copy link
Member

alexanderkiel commented Jun 17, 2021

Because the difference between ge/gt and le/lt is unclear, I just reuse the implementation of ge/le also for gt/lt. This can be done, because it's ok to return more results as would match with a more exact implementation.

The exact implementation semantics will be discovered in #412.

@alexanderkiel alexanderkiel self-assigned this Jun 17, 2021
@alexanderkiel alexanderkiel added enhancement New feature or request module:db labels Jun 17, 2021
@alexanderkiel alexanderkiel changed the title Implement Prefixes gt and lt for Date Search Params Use Implementation of ge/le for gt/lt in Date Search Params Jun 17, 2021
@alexanderkiel alexanderkiel added this to the v0.11 milestone Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request module:db
Projects
None yet
Development

No branches or pull requests

1 participant