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.

Sign up for GitHub

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

Jump to bottom

Query polish #698

Closed
wants to merge 305 commits into from
Closed

Query polish #698

wants to merge 305 commits into from

Conversation

philvarner
Copy link
Collaborator

Related Issue(s): #692

Proposed Changes:

  1. Update YAML types to reflect what's being done in practice
  2. Update examples to be correct wrt spec

These are technically breaking changes, but I think they more accurately reflect the handful of implementations of this.

PR Checklist:

  • [] This PR has no breaking changes.
  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.
  • API only: I have run npm run generate-all to update the generated OpenAPI files.

Phil Varner and others added 30 commits November 7, 2019 10:51
update openapi definition wrt non-json format for get parameter
7695ce3
@matthewhanson @joshfix
Search context (radiantearth#633)
1fdaf20 
* rename search api extension to context

* update OpenAPI yaml
@m-mohr
Merge branch 'master' into asset-purpose-field
679a394
@anayeaye
defines stac_extensions and requires stac_version for itemcollection
74ed979
update sortby yaml to be a string
dba1ac1
update sortby yaml to be a string
248893e
@anayeaye
Merge branch 'dev' of https://github.com/radiantearth/stac-spec into …
e4a63fd 
…item-collection-version-and-extensions

Merge upstream dev into branch
Merge branch 'dev' into sort
e58f0aa
update the name of this field to be rel
f9529b0
@anayeaye
corrected itemcollection readme
9a9de46
@jbants
addressed comments
56cc65c
@jbants
Merge branch 'dtr_extension' of https://github.com/jbants/stac-spec i…
a1e45c5 
…nto dtr_extension
@jbants
Merge branch 'dev' of https://github.com/radiantearth/stac-spec into …
ff21540 
…dtr_extension
@scisco
address PR comments
a5ce514
@scisco
merge with dev
fd82ad3
@anayeaye
no changes were needed to itemcollection schema which references tthe…
0b38d04 
… item schema (where stac_version and stac_extensions are already defined)
@scisco
add best practices for static versioning
d16c886
@scisco
fix numbering
0313e84
@m-mohr
Version extension created, added deprecation, removed version from co…
3d172a3 
…llection.
@m-mohr
Removed version also from API.
675a062
@anayeaye
revert item collection schema
44f9a0c
@anayeaye
more corrections to itemcollection.md
c444e2c
@scisco
fix paths
45d4579
@matthewhanson
initial sat extension
ef0d6ad
@anayeaye
updated itemcollection exmaples
fc9c22b
@scisco
change rel types to a table
e99f508
@matthewhanson
updated sat fields
71a60f4
@scisco
fix typo
b8cc498
fixing examples to include method (radiantearth#645)
1aa1836
@jbants
Merge branch 'dev' of https://github.com/radiantearth/stac-spec into …
6998cbb 
…dtr_extension
matthewhanson and others added 29 commits November 26, 2019 12:31
@matthewhanson
Merge pull request radiantearth#670 from radiantearth/sar_changes
a2f52d6 
SAR Extension changes
@m-mohr
Merge branch 'dev' into catalog-spec-cleanup
07403b1
@m-mohr
Validate SAR and EO asset extensions correctly.
7ba4c9f
@m-mohr
Merge pull request radiantearth#684 from radiantearth/catalog-spec-cl…
15b3fc2 
…eanup

Clean-up catalog relations and links
@m-mohr
Merge branch 'dev' into eo-asset-schema
6f795e9
Merge branch 'dev' into asset-purpose-field
996e5d6
fix changelog
80e1306
fix roles json schema
1455eb9
update asset extension role to roles
95ca274
update other uses of asset role to roles
eb27865
@m-mohr
Merge pull request radiantearth#655 from philvarner/asset-purpose-field
1e30353 
allow multiple roles in asset
@matthewhanson
Merge branch 'dev' into eo-asset-schema
74d00c9
@matthewhanson
Merge pull request radiantearth#686 from radiantearth/eo-asset-schema
c88571d 
Validate SAR and EO asset extensions correctly.
@matthewhanson
Merge branch 'dev' into version_best_practice
6aa7d39
@scisco
Merge branch 'dev' into version_best_practice
f978ed0
@scisco
resolve PR comments
19e03c9
@scisco
Merge branch 'version_best_practice' of github.com:radiantearth/stac-…
9f96fad 
…spec into version_best_practice
@davidraleigh
link to python gRPC STAC client
c5e8774
@davidraleigh
Merge branch 'dev' into issue-575
57422ce
@matthewhanson
Merge pull request radiantearth#668 from davidraleigh/issue-575
9732e52 
Issue 575
@matthewhanson
Merge branch 'dev' into version_best_practice
d3cf35a
@scisco
update links
594d3dc
@scisco
Merge pull request radiantearth#641 from radiantearth/version_best_pr…
c12e37e 
…actice

add best practices for static versioning
@scisco
update schema for bbox which fixes radiantearth#587
bfdb884
@scisco
update changelog and run yaml merge
47aff05
@m-mohr
Moved STAC specific wording from OAFeat.yaml to STAC.yaml, added code…
e3f8a2a 
… fences.
@m-mohr
Merge pull request radiantearth#695 from radiantearth/587_remove_brac…
cedff6d 
…kets_bbox

update schema for bbox which fixes radiantearth#587
query extension types are more accurate
24cc8e5
add query extension changes to Changelog
0fcf234
@philvarner philvarner closed this Dec 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@philvarner @m-mohr @matthewhanson @anayeaye @jbants @scisco @nrweir @cholmes @davidraleigh @jisantuc