Skip to content

Conversation

@m-mohr
Copy link
Collaborator

@m-mohr m-mohr commented Oct 15, 2019

Related Issue(s): None

Proposed Changes:

  1. It's not a bug, but the oneOf is not needed in the schema. Therefore removed it for simplicity.
  2. Removed the "geometry" and "type" as required property as it is already required in the GeoJSON Feature schema (and we do not redefine it in the schema, therefore it is referring to a non-existing property anyway).
  3. Added "bbox" as property so that the "require" flag really works.
  4. Fixed a typo: "updated"->"created"

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.

@m-mohr m-mohr added this to the 0.9.0 milestone Oct 15, 2019
@m-mohr m-mohr modified the milestones: 0.9.0, 0.8.1 Oct 25, 2019
@matthewhanson matthewhanson merged commit 91d6478 into dev Oct 25, 2019
@matthewhanson matthewhanson deleted the item-schema-fixes branch October 25, 2019 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport backport to stable stac-item

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants