Skip to content

fix(docs): host README screenshots in the repository - #71

Merged
rboni-dk merged 1 commit into
mainfrom
fix/readme-screenshots
Aug 26, 2026
Merged

fix(docs): host README screenshots in the repository#71
rboni-dk merged 1 commit into
mainfrom
fix/readme-screenshots

Conversation

@rboni-dk

Copy link
Copy Markdown
Contributor

The three feature screenshots in the README were hot-linked to
datakitchen.io's WordPress media library. All three now return 404 — the
media was removed or renamed on the website side — so the README shows
broken images on both the GitHub project page and the PyPI project page,
which publishes this README as the package long description.

GitHub also wraps README images in an anchor to its image proxy, so each
broken image renders as blue underlined alt text that leads to a
"Not Found" page when clicked. Readers see three broken images and three
dead links.

This commits the recovered originals to docs/assets/images/readme/ and
references them by absolute raw-content URL. Repository-relative paths
would fix the GitHub page but not PyPI, which cannot resolve them.

Two of the original filenames contained a narrow no-break space (U+202F)
from macOS screenshot naming — invisible in the source, and a hazard for
any restore under the same names. The new names are ASCII-only.

Also fixes the DatKitchen typo in the first image's alt text.

The three feature screenshots were hot-linked to the marketing site's
media library, which now returns 404 for all three, so the README shows
broken images on both the GitHub project page and PyPI. GitHub also wraps
README images in an anchor to its image proxy, so each broken image
renders as a clickable link to a "Not Found" page.
Commit the recovered originals under ASCII-only names and reference them
by absolute raw-content URL. Two of the original filenames contained a
narrow no-break space (U+202F) from macOS screenshot naming, invisible in
the source; the new names avoid it. Repository-relative paths would fix
the GitHub page but not PyPI, which cannot resolve them.
@github-actions

Copy link
Copy Markdown

Coverage

Coverage Report •
FileStmtsMissBranchBrPartCoverMissing
TOTAL23336173056124420%
report-only-changed-files is enabled. No changed files were found in the coverage report :)

TestsSkippedFailuresErrorsTime
40 💤0 ❌4 🔥19.119s ⏱️

Sign up for freeto 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

@rboni-dk@aarthy-dk