Skip to content

docs: update Cloud Capture image - #426

Merged
gsavage merged 2 commits into
mainfrom
new-image-for-cloud-capture
Sep 25, 2026
Merged

gsavage merged 2 commits into
mainfrom
new-image-for-cloud-capture

Conversation

@gsavage

@gsavage gsavage commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

JC has drawn a much nicer Cloud Capture image, so I have replaced my ugly one with his.

@gsavage
gsavage requested a review from a team as a code owner September 25, 2026 12:41
@mintlify

mintlify Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
kosli 🟢 Ready View Preview Sep 25, 2026, 1:14 PM

@mbevc1 mbevc1 changed the title Update Cloud Capture image docs: update Cloud Capture image Sep 25, 2026
mbevc1
mbevc1 previously approved these changes Sep 25, 2026
@claude

claude Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @gsavage's task in 1m 16s —— View job


Doc review

administration/cloud_capture/overview.md:23 — Improvement
The alt text was not updated alongside the image it describes. It still reads "sending queries to and receiving snapshots from three customer cloud accounts, then passing the data to the Kosli API and database", but the new diagram shows one-way arrows into the three customer clouds over a link labelled IAM / Read Only, and a single Kosli datastore — there is no API element, and the read-only qualifier (the security point this page makes in both the Overview and Security sections) never reaches a screen-reader user.

<img src="/images/administration/cloud-capture-overview.png" />

Fix this →

images/administration/cloud-capture-4-steps.png — Suggestion
Removing the <Frame> from the Operation section leaves this file in the repo with no page referencing it (grep across .md/.mdx/.json finds zero hits). Worth a git rm images/administration/cloud-capture-4-steps.png in this PR so it doesn't linger as an unexplained asset.


Links, navigation, and redirects check out — no page was added, moved, or deleted, and the four internal links on the page all resolve. 0 Critical, 1 Improvement, 1 Suggestion; nothing here blocks the merge.
· branch new-image-for-cloud-capture

JC has drawn a much nicer Cloud Capture image, so I have replaced my
ugly one with his.
Remove the image showing the 4-step process of Cloud Capture.  It
doesn't add any value.
@gsavage
gsavage merged commit 5302c51 into main Sep 25, 2026
6 checks passed
@gsavage
gsavage deleted the new-image-for-cloud-capture branch September 25, 2026 13:28

This branch was successfully deployed

1 active deployment
staging — 221d4c1e Deployed Sep 25, 2026 by mintlify[bot]
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