Skip to content

fix: fall back to img attribute when image is missing#223

Merged
DoDoIndex merged 1 commit intodevelopfrom
fix/img-fallback
Feb 12, 2026
Merged

fix: fall back to img attribute when image is missing#223
DoDoIndex merged 1 commit intodevelopfrom
fix/img-fallback

Conversation

@paulgnz
Copy link
Contributor

@paulgnz paulgnz commented Feb 12, 2026

Summary

  • Some NFT collections use img instead of the standard image attribute for IPFS CIDs in their schema
  • This caused the marketplace to show blank images with GET /ipfs/undefined errors
  • Added normalizeTemplate() in services/templates.ts that maps imgimage at the data layer
  • Applied to all template fetch functions so every component works without changes
  • Also added img fallback in SearchInputResultsList for search results

Test plan

  • Verify collections using image attribute still display correctly
  • Verify collections using img attribute now display correctly (e.g. charlieart12 on testnet)
  • Test search results show images for both attribute names

🤖 Generated with Claude Code

Some collections use 'img' instead of the standard 'image' attribute
for IPFS CIDs. Add normalizeTemplate() to map img → image at the
data layer so all components work without changes.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@netlify
Copy link

netlify bot commented Feb 12, 2026

Deploy Preview for dev-protonmarket-com ready!

Name Link
🔨 Latest commit 744a891
🔍 Latest deploy log https://app.netlify.com/projects/dev-protonmarket-com/deploys/698e4c780b72bd0007a55c63
😎 Deploy Preview https://deploy-preview-223--dev-protonmarket-com.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@DoDoIndex DoDoIndex merged commit 11213db into develop Feb 12, 2026
5 checks passed
@DoDoIndex DoDoIndex deleted the fix/img-fallback branch February 12, 2026 22:01
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