Skip to content

Conversation

philvarner
Copy link
Collaborator

@philvarner philvarner commented Oct 13, 2022

Related Issue(s):

  • n/a

Proposed Changes:

  1. Release 1.0.0-rc.2

https://github.com/radiantearth/stac-api-spec/compare/v1.0.0-rc.1..main

Changelog

Added

  • Added optional numberMatched and numberReturned fields to ItemCollection to align with OGC Commons
    and OAFeat.

Changed

  • The Collections specification has been incorporated into the Features specification, but remains as
    a separate conformance class.
  • The Browseable specification has been incorporated into the Core specification, but remains as
    a separate conformance class.
  • Extensions moved to standalone specification repositories:

Fixed

  • Item Search limit parameter semantics now align with OAFeat. The server must not return more Items than the limit and a limit value higher than advertised in the
    service description must return a 400 Bad Request status code.

PR Checklist:

  • This PR has no breaking changes.
  • This PR does not make any changes to the core spec in the stac-spec directory (these are included as a subtree and should be updated directly in radiantearth/stac-spec)
  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

@philvarner philvarner marked this pull request as ready for review October 13, 2022 15:00
@philvarner philvarner requested review from cholmes, matthewhanson, jbants, lossyrob and m-mohr and removed request for cholmes October 13, 2022 15:01
Copy link
Collaborator

@cholmes cholmes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Left one little comment, that wasn't a change on this PR but was highlighted here.

Copy link
Contributor

@jbants jbants left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

m-mohr
m-mohr previously requested changes Oct 24, 2022
Copy link
Collaborator

@m-mohr m-mohr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are still some rc.1 references I think (overview.md -> Conformance Class Table)

@philvarner
Copy link
Collaborator Author

There are still some rc.1 references I think (overview.md -> Conformance Class Table)

Odd, not sure why those didn't get replaced. I will fix.

@philvarner
Copy link
Collaborator Author

@m-mohr fixed the references in overview.md

@philvarner philvarner dismissed m-mohr’s stale review October 25, 2022 00:04

issues resolved

@philvarner philvarner requested a review from m-mohr October 25, 2022 00:21
@philvarner philvarner merged commit d4f8b4a into radiantearth:main Nov 1, 2022
@philvarner philvarner deleted the pv/1.0.0-rc.2-release branch November 1, 2022 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants