Skip to content

fix: point Tyler Gibbs testimonial at his current avatar - #133

Merged
starc007 merged 1 commit into
starc007:mainfrom
chakravartiavinit:fix/testimonial-avatar
Jul 31, 2026
Merged

fix: point Tyler Gibbs testimonial at his current avatar#133
starc007 merged 1 commit into
starc007:mainfrom
chakravartiavinit:fix/testimonial-avatar

Conversation

@chakravartiavinit

@chakravartiavinit chakravartiavinit commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Issue

The Tyler Gibbs testimonial on the landing page renders a broken-image icon instead of an avatar.

TestimonialCard hotlinks the author's pbs.twimg.com profile image. Twitter mints a new profile-image path whenever a user changes their picture and lets the old path 404 — Tyler changed his, so the stored URL

https://pbs.twimg.com/profile_images/2068918593962536960/gmAsn6Nj_normal.jpg

now returns 404 in every size variant (_normal, _bigger, _200x200, bare). The card renders <img> with no error fallback, so the dead URL shows as the browser's broken-image glyph next to the name and handle.

Fix

Point the entry at his current avatar (2077596476687998978/WvlUT0v1). Verified 200 both as stored (_normal) and as rendered — TestimonialCard rewrites _normal_bigger.

Checks

  • Swept every avatar URL in testimonials-data.ts on main in its rendered form; Tyler's is the only dead one.
  • bun run typecheck and bun run lint clean (lint's 3 noImgElement warnings in attachment-upload.tsx are pre-existing and untouched).

Not done

No onError fallback in TestimonialCard — it would make the card a client component for a failure mode that has hit once. Worth adding if a second avatar rots. Migrating avatars to unavatar.io was considered and dropped: it 403'd on one handle during testing and adds ~1s per image.

Screenshot 2026-07-31 at 7 52 33 PM

@starc007
starc007 merged commit df472b8 into starc007:main Jul 31, 2026
1 of 2 checks passed
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