Skip to content

feat: implement reusable nft display#621

Closed
xx7412421-cloud wants to merge 2 commits into
Commitlabs-Org:masterfrom
xx7412421-cloud:grantfox-577-nft-display
Closed

feat: implement reusable nft display#621
xx7412421-cloud wants to merge 2 commits into
Commitlabs-Org:masterfrom
xx7412421-cloud:grantfox-577-nft-display

Conversation

@xx7412421-cloud

Copy link
Copy Markdown
Contributor

Closes #577.

Summary

  • replace the placeholder NFTDisplay with a reusable, prop-driven commitment NFT card
  • render metadata rows, artwork with branded fallback, compliance/maturity highlights, and attestation-history link
  • cover metadata parsing, prop overrides, missing metadata, missing images, and broken image fallback

Validation

  • npm exec -- vitest run src/components/tests/NFTDisplay.test.tsx
  • npm exec -- eslint src/components/NFTDisplay.tsx src/components/tests/NFTDisplay.test.tsx
  • npm exec -- prettier --check src/components/NFTDisplay.tsx src/components/tests/NFTDisplay.test.tsx
  • git diff --check

@vercel

vercel Bot commented Jun 21, 2026

Copy link
Copy Markdown

@xx7412421-cloud is attempting to deploy a commit to the 1nonly's projects Team on Vercel.

A member of the Team first needs to authorize it.

@1nonlypiece

Copy link
Copy Markdown
Contributor

reusable nft display is a good one to tackle, but #577 isn't assigned to you — our process needs you assigned before the PR goes up. closing as invalid; please claim it on the board and re-raise, happy to review then. https://discord.gg/WV7tdYkJk

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.

Implement live NFTDisplay component for Commitment NFTs with metadata and image rendering

2 participants