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.

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

Use the type field to determine if ingest is collection or item #492

Merged
merged 2 commits into from
May 9, 2023

Conversation

jwalgran
Copy link
Collaborator

@jwalgran jwalgran commented May 8, 2023

Related Issue(s):

Proposed Changes:

  1. The type field is required in STAC API 1.0.0 and using that field is preferable to a heuristic based on other field values.

PR Checklist:

  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

@jwalgran jwalgran requested a review from philvarner May 8, 2023 17:48
@jwalgran jwalgran force-pushed the jcw/use-type-when-ingesting branch from 7dd0587 to 94a37ed Compare May 8, 2023 17:55
CHANGELOG.md Outdated Show resolved Hide resolved
Base automatically changed from jcw/remove-stream-based-ingest to main May 8, 2023 20:14
jwalgran and others added 2 commits May 8, 2023 13:28
The `type` field is required in STAC API 1.0.0 and using that field is
preferable to a heuristic based on other field values.

Connects #486
Co-authored-by: Phil Varner <pvarner@element84.com>
@jwalgran jwalgran force-pushed the jcw/use-type-when-ingesting branch from ec0dc9e to 0f625ed Compare May 8, 2023 20:28
@jwalgran
Copy link
Collaborator Author

jwalgran commented May 8, 2023

Thanks for the text correction on the CHANGELOG, @philvarner. I accepted that change and rebased this PR on top of main

@jwalgran
Copy link
Collaborator Author

jwalgran commented May 9, 2023

Thanks for the review.

@jwalgran jwalgran merged commit 859d280 into main May 9, 2023
@jwalgran jwalgran deleted the jcw/use-type-when-ingesting branch May 9, 2023 15:54
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.

2 participants