Open tjellicoe-tpzuk opened 1 month ago
@tjellicoe-tpzuk thanks for the updates, as mentioned in https://github.com/stac-utils/stac-fastapi/pull/684#discussion_r1627290500 we should try to avoid adding more extension
related code either in stac_fastapi.types
and stac_fastapi.api
.
could you try to refactor this PR and move most of the things to the .extensions
submodule? 🙏
Related Issue(s): #615
Description: Adding Collection Search Extension as defined here. Including adding spatiotemporal searching to the GET
/collections
endpoint and adding a POST/collections
endpoint with the same functionality. Also adding alimit
parameter to specify the maximum number of results returned from the search. Note, there seems to be an issue with the test_client_openapi test now due to the check for "Collections" in the schema. I am not sure what is causing this issue, so any help would be much appreciated. This means themake test
fails on this test, although all other tests pass. Some other formatting changes look to have been made by the pre-commit hooks.PR Checklist:
pre-commit
hooks pass locallymake test
) (yes but see comment regarding test_client_openapi test above)make docs
)