docs(showcase): repair broken v2 asset paths in index.html#101
Merged
Conversation
After the CV v2 layered migration (1.6.x), every preset's screenshot and PDF was renamed with a '-v2' suffix on disk (cv-blue-banner-v2.png, cover-letter-executive-v2.pdf, etc.). The docs/index.html GitHub Pages site still referenced the pre-migration paths (without -v2), so 20 of 27 showcase references rendered as 404 — almost the entire CV gallery and the cover-letter previews.
Surfaced by an external visual-presentation audit ahead of the v1.6.6 Maven Central debut: a broken-image-laden showcase site would have been the first impression for every new Central caller.
Fix is mechanical:
cv/cv-X.{pdf,png} -> cv/cv-X-v2.{pdf,png} (14 CV PDFs + 1 PNG)
coverletter/cover-letter-X.{pdf,png} -> coverletter/cover-letter-X-v2.{pdf,png} (4 specific letters)
coverletter/cover-letter.pdf -> coverletter/cover-letter-modern-professional-v2.pdf (the generic 'cover letter' link now points at the recommended preset)
Invoice + proposal + schedule paths were already correct (no v2 migration there).
Verification: all 26 remaining unique references resolve on disk (one collapsed by sorted-unique dedup; same file). No regression to other docs surfaces.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
External presentation audit ahead of the v1.6.6 Maven Central debut surfaced 20 of 27 showcase references in
docs/index.htmlrendering as 404. After the CV v2 layered migration in 1.6.x, every preset's screenshot and PDF was renamed with a-v2suffix on disk, but the GitHub Pages site (demchaav.github.io/GraphCompose/) still referenced the pre-migration paths.Fix
cv/cv-X.{pdf,png}cv/cv-X-v2.{pdf,png}coverletter/cover-letter-X.{pdf,png}coverletter/cover-letter-X-v2.{pdf,png}coverletter/cover-letter.pdfcoverletter/cover-letter-modern-professional-v2.pdfVerification
Why this is a release-blocker
The next Maven Central debut tag (v1.6.6) will route significantly more traffic at the Pages site. Broken-image-laden first impression for every new caller would have cost trust. Fixing this before cut is cheap; fixing after means days of broken homepage in the Central debut window.
This is Phase A of the pre-release polish plan; Phases B + C (hero PDF + README banner replacement) follow.
Test plan
lssedsubstitution only)