-
Notifications
You must be signed in to change notification settings - Fork 47
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
v.1.0.0-rc.1 release merge to master #291
Merged
Merged
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
32c3d83
Filter Extension - add Accent and Case-insensitive Comparison conform…
philvarner 90f848e
require children endpoint to return all child link objects
philvarner 30a54ae
clarify wording of search link rel requirement
philvarner 7fabce4
update example urls to stac-api.example.com (#264)
philvarner a06becf
add type=application/geo+json to prev and next links (#266)
philvarner 255034c
Undeprecate query extension (#262)
philvarner 5753396
update CQL2 definition to reference OGC API yaml file in github, upda…
philvarner 56cd33b
Merge branch 'dev' into children-reflect-root-child-links
philvarner 137502e
Merge pull request #265 from philvarner/children-reflect-root-child-l…
philvarner 1dbbdd0
explicitly state /children may return fewer fields for its entities
722e3ba
update CQL2 implemenation suggestions (#269)
philvarner 2ee0f63
Merge branch 'dev' into pv/allow-subset-of-cat-or-coll-for-children
ad2c2a0
Merge pull request #273 from radiantearth/pv/allow-subset-of-cat-or-c…
philvarner 10636ea
Merge pull request #267 from philvarner/clarify-search-link-rel
philvarner c5d1b2b
change all uses of shall to must, and stylize all non-linked conforma…
philvarner 9a0b8c6
add changelog entry
philvarner 73d907e
update version from 1.0.0-beta.5 to 1.0.0-rc.1
philvarner 87ba737
fix uri for filter extension to remove item-search-filter
philvarner 5e86a63
Merge pull request #276 from philvarner/shall-to-must-and-italics
philvarner e99bfe7
Merge branch 'dev' into pv/filter-extension-conformance-class-consist…
philvarner b6f1ef3
Merge pull request #279 from philvarner/pv/filter-extension-conforman…
philvarner 07fe9b5
Merge branch 'dev' into pv/1.0.0-beta.5-to-rc.1
philvarner 51fdd16
fix merge problem
philvarner a965195
update maturity levels in preparation for 1.0.0-rc.1
philvarner 979182e
Merge pull request #278 from philvarner/pv/1.0.0-beta.5-to-rc.1
philvarner eb98eb2
Merge branch 'dev' into update-maturity
philvarner 3abd06a
Add Versions section to README
philvarner c42290a
add all conformance class links to deployed index
philvarner b0971e5
fix a few minor openapi spec issues, upgrade remark and spectral
4519746
re-enable max line lenght
b28a87c
convert external spectral config into package.json
d2eb437
remove unused openapi spec elements and fix several stac_version values
d06b1e3
remove stray test from Filter
8a846d1
add remark-gfm plugin to fix issue with false positives on long lines…
ac16ed6
reformat
c29de7a
Merge pull request #283 from philvarner/update-maturity
philvarner 0d4d0ca
Merge pull request #290 from radiantearth/pv/remove-stray-text
philvarner a5a380c
Merge pull request #284 from philvarner/pv/add-versions-to-readme
philvarner a81b34c
Merge pull request #289 from radiantearth/remove-unused-openapi-spec-…
philvarner 9ef192c
Merge branch 'dev' into openapi-cleanup-and-spectral-and-remark-upgrade
philvarner 0c36fce
format tables in extensions
philvarner b14ec82
Merge pull request #287 from radiantearth/openapi-cleanup-and-spectra…
philvarner 59c8c1a
Claify Transaction Extension behavior (#271)
philvarner e1f2910
update changelog v1.0.0-rc.1 release date
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 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
This file contains 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
This file was deleted.
Oops, something went wrong.
This file contains 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 |
---|---|---|
@@ -1,5 +1,6 @@ | ||
**Related Issue(s):** # | ||
**Related Issue(s):** | ||
|
||
- # | ||
|
||
**Proposed Changes:** | ||
|
||
|
This file contains 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
This file contains 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 @@ | ||
16 |
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
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.
does this mean the geojson feature has to have an id field?
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.
good question -- I had thought geojson feature required id, but now I see it's a "should". So maybe we need to clarify that behavior -- maybe just say that it's up to the implementer if they want to autogenerate IDs.
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.
Decided the answer to this is "yes". The openapi spec already defined
id
as a required field.