Skip to content

Update large-assets.md#130

Open
CaptainInler wants to merge 2 commits intogeoadmin:masterfrom
CaptainInler:master
Open

Update large-assets.md#130
CaptainInler wants to merge 2 commits intogeoadmin:masterfrom
CaptainInler:master

Conversation

@CaptainInler
Copy link
Copy Markdown

Change "How It Works" to better distinguish between the probe and download steps

Change "How It Works" to better distinguish between the probe and download steps
@ltclm ltclm self-requested a review March 30, 2026 10:00
Copy link
Copy Markdown
Contributor

@ltclm ltclm left a comment

Choose a reason for hiding this comment

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

Thanks for improving the structure!
Breaking out the probe and download steps makes the flow much clearer.

Quick note: there's a typo in line 9 — "chuncks" should be "chunks".

@ltclm ltclm self-requested a review March 30, 2026 18:37
@ltclm
Copy link
Copy Markdown
Contributor

ltclm commented Mar 30, 2026

Thanks for fixing the typo! There are some formatting issues left that need to be addressed. Could you please run the following commands in your local project:

npm install
npm run format

Then commit and push these changes. After that, we'll be ready to merge.

@CaptainInler
Copy link
Copy Markdown
Author

CaptainInler commented Mar 31, 2026

Thanks for fixing the typo! There are some formatting issues left that need to be addressed. Could you please run the following commands in your local project:

npm install
npm run format

Then commit and push these changes. After that, we'll be ready to merge.

@ltclm I do not have npm installed.. Is this something you could do instead?

@ltclm
Copy link
Copy Markdown
Contributor

ltclm commented Mar 31, 2026

@ltclm I do not have npm installed.. Is this something you could do instead?

@CaptainInler, I’ve formatted the file for you. You can find the changes in this PR: #131

We can either merge this PR directly, or you can update your fork and we’ll merge from there. Let me know which you prefer!

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