Skip to content

Extract shared ItemCard with value and date - #35

Merged
amrtgaber merged 1 commit into
mainfrom
refactor/shared-item-card
Feb 7, 2026
Merged

Extract shared ItemCard with value and date#35
amrtgaber merged 1 commit into
mainfrom
refactor/shared-item-card

Conversation

@amrtgaber

Copy link
Copy Markdown
Contributor

Summary

  • Extract ItemCard into a shared component at src/components/item-card.tsx
  • Both the All Items page and collection detail page now use the same card
  • Cards consistently show: estimated value, date added, collection name badge, tag badges, and condition badge

Test plan

  • pnpm lint + pnpm test:run + pnpm build all pass
  • Manual: verify item cards on collection detail page now show value and date
  • Manual: verify item cards on All Items page still look correct
  • Manual: collection name badge only appears on All Items page (since collection detail items don't have collection_name populated differently)

Move ItemCard into its own component at src/components/item-card.tsx
and use it in both the All Items page and collection detail page.
Cards now consistently show estimated value, date added, collection
name badge, tag badges, and condition badge everywhere.
@netlify

netlify Bot commented Feb 7, 2026

Copy link
Copy Markdown

Deploy Preview for trove-web-production ready!

Name Link
🔨 Latest commit 93c03de
🔍 Latest deploy log https://app.netlify.com/projects/trove-web-production/deploys/6986866cb149880008d986a1
😎 Deploy Preview https://deploy-preview-35--trove-web-production.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.

@amrtgaber
amrtgaber merged commit f2a06ac into main Feb 7, 2026
6 checks passed
@amrtgaber
amrtgaber deleted the refactor/shared-item-card branch February 7, 2026 00:30
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.

1 participant