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

fix: Item defined 'properties' as a required field, but didn't includ… #231

Merged
merged 1 commit into from
Dec 1, 2021
Merged

fix: Item defined 'properties' as a required field, but didn't includ… #231

merged 1 commit into from
Dec 1, 2021

Conversation

philvarner
Copy link
Collaborator

Related Issue(s): # n/a

Proposed Changes:

  1. add missing properties properties entry to Item. It was defined as a required field, but not actually in the properties definition.

PR Checklist:

  • This PR is made against the dev branch (all proposed changes except releases should be against dev, not master).
  • 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 requested a review from m-mohr November 11, 2021 14:47
@philvarner philvarner merged commit 1350306 into radiantearth:dev Dec 1, 2021
@philvarner philvarner deleted the add-properties-as-property-of-item branch December 1, 2021 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-mohr m-mohr m-mohr approved these changes

@geospatial-jeff geospatial-jeff geospatial-jeff approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@philvarner @m-mohr @geospatial-jeff