Skip to content

STAC Common Metadata #682

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
Merged
matthewhanson merged 9 commits into from
Dec 20, 2019
Merged

STAC Common Metadata #682

matthewhanson merged 9 commits into from
Dec 20, 2019

Conversation

@m-mohr
Copy link
Collaborator

@m-mohr m-mohr commented Nov 25, 2019 •
edited
Loading

Related Issue(s): #659

Proposed Changes:

  1. Made centralized list of common metadata fields.
  2. To be discussed: intruments and platform are not required any longer.

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 Nov 25, 2019
@m-mohr m-mohr changed the title Common metadata STAC Common Metadata Nov 25, 2019
@m-mohr m-mohr mentioned this pull request Nov 26, 2019
Closed
@m-mohr m-mohr marked this pull request as ready for review December 19, 2019 15:14
scisco
scisco reviewed Dec 19, 2019
View reviewed changes
@m-mohr
Copy link
Collaborator Author

m-mohr commented Dec 20, 2019

Thanks @scisco for the review. I fixed all the raised issues.

@m-mohr m-mohr requested a review from scisco December 20, 2019 11:17
@matthewhanson matthewhanson merged commit 51ddf47 into dev Dec 20, 2019
@matthewhanson matthewhanson deleted the common-metadata branch December 20, 2019 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@matthewhanson matthewhanson matthewhanson approved these changes

@philvarner philvarner Awaiting requested review from philvarner

@anayeaye anayeaye Awaiting requested review from anayeaye

@cholmes cholmes Awaiting requested review from cholmes

@hgs-msmith hgs-msmith Awaiting requested review from hgs-msmith

@jbants jbants Awaiting requested review from jbants

@joshfix joshfix Awaiting requested review from joshfix

+1 more reviewer

@scisco scisco scisco approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

stac-item

Projects

None yet

Milestone

0.9.0-RC1

Development

Successfully merging this pull request may close these issues.

4 participants

@m-mohr @matthewhanson @scisco