-
Notifications
You must be signed in to change notification settings - Fork 27
Add filter extension #825
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
Merged
philvarner
merged 30 commits into
stac-utils:main
from
pjhartzell:pjh/feature/add-filter-extension
Jan 24, 2025
Merged
Add filter extension #825
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
a6b1374
feat: add filter extension to conformance
pjhartzell fb1a000
feat: add cql2 filter extension for POST
pjhartzell 94f6278
fix: remove commented code, get rid of .only in tes
pjhartzell 7306a12
feat: cql2 filter extension for GET
pjhartzell 41ecbf9
fix: use fully qualified property names
pjhartzell 0237061
test: remove redundant test
pjhartzell 9970b68
build: 'npm audit fix' to get CI passing
pjhartzell 647a0a2
feat: check for correct filter-lang and filter-crs
pjhartzell 69d2c3d
feat: Add filter extension to aggregate endpoint
pjhartzell 6cace3a
docs: update CHANGELOG
pjhartzell 5697cf0
test: add test for top level field and property in filter extension f…
pjhartzell 285c994
review: move constants out of function
pjhartzell 80243d4
review: stacQlQuery -> stacqlQuery
pjhartzell 9388cab
review: buildQuery -> buildOpenSearchQuery
pjhartzell f95a102
review: Add minimum_should_match = 1 for 'should' clauses
pjhartzell cf61e72
fix: remove .only from test, update a test comment
pjhartzell dcbf37d
review: remove explicit prefixing of properties object fields
pjhartzell 297d10b
feat: raise error when collection queryables has additionalProperties…
pjhartzell 21cd5a4
style: align existing test format
pjhartzell b53f3d6
feat: add query and filter extensions to openapi.yaml
pjhartzell 2eb8de1
docs: update README.md
pjhartzell 48e507d
fix: finish adding filters extension to the OpenAPI spec
pjhartzell a948319
fix: minor changes to query extension in openapi.yaml
pjhartzell b5b16c6
docs: augument CHANGELOG entry, tweak extension list in README
pjhartzell 74c3d14
review: remove bbox from UNPREFIXED_FIELDS
pjhartzell dafa26d
review: clarify that fields **must not** be prefixed
pjhartzell 25ee9d3
review: add deprecated note to Context Extension
pjhartzell b3a8f78
review: Add note about filter expression terms exactly matching Item …
pjhartzell adc6110
review: add back bbox and note that bbox and geometry are not yet sup…
pjhartzell 5d81274
review: add back bbox
pjhartzell File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
41 changes: 41 additions & 0 deletions
41
tests/fixtures/stac/collection-with-incorrect-queryables.json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,41 @@ | ||
| { | ||
| "id": "landsat-8-l1-incorrect-queryables", | ||
| "type": "Collection", | ||
| "stac_version": "1.0.0", | ||
| "description": "Landat-8 L1 Collection-1 imagery radiometrically calibrated and orthorectified using gound points and Digital Elevation Model (DEM) data to correct relief displacement.", | ||
| "links": [], | ||
| "stac_extensions": [], | ||
| "title": "Landsat-8 L1 Collection-1", | ||
| "extent": { | ||
| "spatial": { | ||
| "bbox": [ | ||
| [ | ||
| -180, | ||
| -90, | ||
| 180, | ||
| 90 | ||
| ] | ||
| ] | ||
| }, | ||
| "temporal": { | ||
| "interval": [ | ||
| [ | ||
| "2013-06-01T00:00:00Z", | ||
| null | ||
| ] | ||
| ] | ||
| } | ||
| }, | ||
| "queryables": { | ||
| "$schema": "https://json-schema.org/draft/2020-12/schema", | ||
| "$id": "", | ||
| "type": "object", | ||
| "title": "", | ||
| "properties": { | ||
| "eo:cloud_cover": { | ||
| "$ref": "https://stac-extensions.github.io/eo/v1.0.0/schema.json#/definitions/fields/properties/eo:cloud_cover" | ||
| } | ||
| }, | ||
| "additionalProperties": false | ||
| } | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.