-
Notifications
You must be signed in to change notification settings - Fork 13
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 a stable sort by default for all queries for Items #52
Conversation
Revert "revert mappings back to original ones" This reverts commit 317a99c.
…etime,id,collection
@@ -1,7 +1,7 @@ | |||
"""Item crud client.""" | |||
import json | |||
import logging | |||
from datetime import datetime | |||
from datetime import datetime as datetime_type |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixes a warning about variables shadowing outer scope
Search() | ||
.using(self.client) | ||
.index("stac_items") | ||
.sort( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
applies a sort now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice
Related Issue(s):
Description:
Previously, no default sort was used. Now, a default sort of properties.datetime,id,collections is used.
PR Checklist:
pre-commit run --all-files
)make test
)make docs
)