Skip to content

Update large-assets.md#131

Merged
ltclm merged 3 commits intomasterfrom
update_large_asset_download_doc
Apr 16, 2026
Merged

Update large-assets.md#131
ltclm merged 3 commits intomasterfrom
update_large_asset_download_doc

Conversation

@ltclm
Copy link
Copy Markdown
Contributor

@ltclm ltclm commented Mar 31, 2026

Change "How It Works" to better distinguish between the probe and download steps.
provided by @CaptainInler

Test link

Change "How It Works" to better distinguish between the probe and
download steps.
provided by @CaptainInler
Copy link
Copy Markdown
Contributor

@asteiner-swisstopo asteiner-swisstopo left a comment

Choose a reason for hiding this comment

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

I like the overview given at the beginning. I would at least fix the typo ("this --> "these") or rephrase certain certain parts for clarity 🙃

Comment thread download-data/stac-api/large-assets.md Outdated
Comment thread download-data/stac-api/large-assets.md
Comment thread download-data/stac-api/large-assets.md Outdated
- Fix intro sentence to use active voice
- Remove "on Linux" qualifier from curl example
- Make section headings consistent with numbered list
- Add section for optional SHA-256 checksum verification
@ltclm ltclm merged commit 8733748 into master Apr 16, 2026
2 checks passed
@ltclm ltclm deleted the update_large_asset_download_doc branch April 16, 2026 08:33
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