Skip to content

fix: use absolute image URLs in README so they render on the registry - #20

Merged
aesslinger merged 1 commit into
mainfrom
fix/readme-absolute-image-urls
Aug 14, 2026
Merged

fix: use absolute image URLs in README so they render on the registry#20
aesslinger merged 1 commit into
mainfrom
fix/readme-absolute-image-urls

Conversation

@aesslinger

Copy link
Copy Markdown
Collaborator

Summary

README's header logo/plus-icon and screenshot-gallery images used repo-relative src paths (assets/plus.svg, assets/screenshots/*.png). GitHub auto-resolves relative image paths against the repo's raw-content base, so they rendered correctly there — but the Tabularium registry serves the README's rendered HTML standalone, with no such resolution. Once the registry's README sync finally caught up (confirmed today, readmeHtml is now populated on /plugins/postgresql), all 6 images 404'd on the actual published plugin page.

Switched all 6 to absolute raw.githubusercontent.com URLs, matching the pattern already used for postgresql-icon.svg (in the header, already absolute) and the .tabularium screenshots array (already absolute, which is why those never had this problem).

Verification

  • npx markdownlint-cli README.md CHANGELOG.md → clean
  • Confirmed no remaining src="assets/... relative paths in the README
  • Absolute URLs match the exact pattern already proven working elsewhere in this file and in .tabularium

Test plan

  • README/CHANGELOG markdownlint-clean
  • No relative asset paths remain
  • Confirm images render correctly on the registry plugin page after this merges and the next refresh/webhook syncs

README's header logo/plus-icon and screenshot-gallery images used
repo-relative src paths. GitHub auto-resolves those against the repo's
raw-content base, so they rendered fine there -- but the Tabularium
registry serves the README's HTML standalone with no such resolution,
so all 6 images 404'd on the plugin's actual registry page once the
README finally synced. Switched to absolute raw.githubusercontent.com
URLs, matching the pattern already used for postgresql-icon.svg and the
.tabularium screenshots array.
@aesslinger aesslinger added the prerelease:beta Version suggestion targets a beta prerelease label Aug 14, 2026
@github-actions

Copy link
Copy Markdown

Version suggestion

Based on this PR's title (fix) and the prerelease:beta label:

Current 1.0.0-beta.5
Suggested next tag v1.0.0-beta.6

This is informational only — no tag or release is created automatically yet.

@aesslinger
aesslinger merged commit 33e9ba9 into main Aug 14, 2026
20 of 21 checks passed
@aesslinger
aesslinger deleted the fix/readme-absolute-image-urls branch August 14, 2026 16:37
aesslinger added a commit that referenced this pull request Aug 14, 2026
Continuation of the beta line after PR #20 (absolute image URLs in
README, fixing broken images on the Tabularium registry page) and the
carried-forward PR #19 description rewrite. Registry-listed release.

Verified: .tabularium re-validated clean against the live registry
schema; cargo build/test (85/85)/clippy/fmt all pass; markdownlint clean.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

prerelease:beta Version suggestion targets a beta prerelease

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant