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

Orders API Client Support For Multiple Fallback Bundle(s) #1106

Merged
merged 10 commits into from
Mar 21, 2025

Conversation

asonnenschein
Copy link
Contributor

@asonnenschein asonnenschein commented Mar 19, 2025

GH Ticket: #1105

This MR adds support in the Orders SDK and CLI for specifying multiple fallback bundles, updates the docs, and updates the orders CLI and SDK tests with success and failure cases for fallback bundles.

CLI example:

planet orders request --item-type "PSScene" \
    --bundle "analytic_8b_udm2" \
    --fallback-bundle "analytic_udm2,analytic_3b_udm2" \
    --name "my order" \
    "20250130_035211_69_2516"

SDK example:

order_request.product(
    ["20250130_035211_69_2516"],
    "analytic_8b_udm2",
    "PSScene",
    "analytic_udm2,analytic_3b_udm2"
)

@asonnenschein asonnenschein changed the title Draft: Orders API Client Support For Multiple Fallback Bundle(s) Orders API Client Support For Multiple Fallback Bundle(s) Mar 20, 2025
Copy link

@ceholden ceholden left a comment

Choose a reason for hiding this comment

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

I think this looks great and will fulfill the feature request, thank you! I had a couple suggestions about docstrings and a suggestion to cover the fallback_bundle: list[str] case in the tests added in this PR, but nothing functional to change. Nice work and thanks for incorporating into the user guide!

@asonnenschein asonnenschein merged commit 4374ec8 into main Mar 21, 2025
9 checks passed
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.

3 participants