-
Notifications
You must be signed in to change notification settings - Fork 98
Implement CLI commands: planet data asset-get, download, activate, wait #864
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
Merged
Merged
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
d71eb89
Added set for the new registered command.
kevinlacaille 0347cbc
Added a todo for the new callback function.
kevinlacaille 3700ed6
Added a skelleton to a needed callback function, along with the outli…
kevinlacaille 03998a8
Added commands: asset-download, get, activate, wait
kevinlacaille fd1077d
A working test for download.
kevinlacaille 9200fcf
Removed commented out examples.
kevinlacaille cfcadbd
tests for activate and wait
kevinlacaille 19a45a0
Docstrings.
kevinlacaille 1bacfea
linting
kevinlacaille 0dddfb2
test for asset-get
kevinlacaille 23264d1
Removed commented out function.
kevinlacaille a246c06
Removed unused tests.
kevinlacaille 7a087ce
removed plural item types.
kevinlacaille 48ab53c
took echo out of context of data_client
kevinlacaille 002b6d5
upgraded readability of CLI test
kevinlacaille f7362af
updated asset-download to follow the useage docs, commented out asset…
kevinlacaille da9c14a
updated asset-download test, commented out asset-get test
kevinlacaille 3dd0ef7
Added callback function to item type in download call
kevinlacaille 3970e1e
write bytes to tif, not text
kevinlacaille f106489
ensured asset-wait and -activate followed the docs.
kevinlacaille 0b3a24d
More elegant version of removing item type from list.
kevinlacaille 689517f
Added asset-get response to wait and activate
kevinlacaille 8aa9be5
moved asset-get response into a fixture
kevinlacaille 60057a0
linting
kevinlacaille b853a2f
Merge branch 'main' into data-asset-download-507
kevinlacaille 9d1739c
fixed merge conflict linting issues
kevinlacaille c995a8c
added back decorator to a fixture, whoopsie
kevinlacaille 6da207d
Added data asset commands to docs.
kevinlacaille f01a519
Added new test for check_item_type, renamed tests for check_item_types
kevinlacaille 5d95584
Added new callback function to validate a single time type and edited…
kevinlacaille cf13ee9
Added BadParameter if more than 1 item type was provided.
kevinlacaille 5574096
Added test if more than 1 item type was provided.
kevinlacaille faf07f5
move return outside of try/except block
kevinlacaille c753e9e
remove badparameter for more than 1 item type
kevinlacaille ee268a7
asset_type_id -> asset_type
kevinlacaille File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.