Skip to content

Conversation

YuriZmytrakov
Copy link
Contributor

@YuriZmytrakov YuriZmytrakov commented Sep 22, 2025

Related Issue(s):

#403

Description:

This PR introduces a new env var USE_DATETIME to control the datetime filtering behavior in the search.

PR Checklist:

  • Code is formatted and linted (run pre-commit run --all-files)
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable
  • Changes are added to the changelog

@YuriZmytrakov YuriZmytrakov marked this pull request as draft September 23, 2025 07:29
@YuriZmytrakov YuriZmytrakov force-pushed the CAT-1413-3 branch 18 times, most recently from 12efb81 to 908c9bd Compare September 23, 2025 14:14
- Added USE_DATETIME env var to control datetime filtering
- USE_DATETIME=True (default): use existing logic that handles both datetime and start/end datetime fields
- USE_DATETIME=False (default): use only start/end datetime fields for search
- Added USE_DATETIME env var tests
@YuriZmytrakov YuriZmytrakov changed the title dummy USE_DATETIME env var for filter behavior Sep 23, 2025
@YuriZmytrakov YuriZmytrakov marked this pull request as ready for review September 23, 2025 14:47
@jonhealy1 jonhealy1 self-requested a review September 23, 2025 16:23
Copy link
Collaborator

@jonhealy1 jonhealy1 left a comment

Choose a reason for hiding this comment

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

Nice work

@jonhealy1 jonhealy1 merged commit 7d6b741 into stac-utils:main Sep 23, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants