docs: update SESSION_STATE for Session 29 - Image Display Bugfix
Documented the image display issue, root cause (type system mismatch), and the fix applied in v1.14.1. Images now correctly display from the auto-photo-save feature in both ItemDetailModal and InventoryTable.
This commit is contained in:
@@ -1,9 +1,46 @@
|
||||
# CURRENT AI WORKING SESSION — HANDOVER
|
||||
|
||||
**Active AI:** Claude Haiku 4.5
|
||||
**Last Updated:** 2026-04-21 (Session 28 - Phase 3 Complete)
|
||||
**Current Version:** v1.14.0 (Phase 3: AI Extraction + Auto-Photo-Save — COMPLETE)
|
||||
**Branch:** dev (All 8 Phase 3 tasks complete, production-ready)
|
||||
**Last Updated:** 2026-04-21 (Session 29 - Image Display Bugfix)
|
||||
**Current Version:** v1.14.1 (Image display fix: photo_path fields now surfaced to frontend)
|
||||
**Branch:** dev (Phase 3 complete + bugfix applied, production-ready)
|
||||
|
||||
---
|
||||
|
||||
## SESSION 29 SUMMARY — Image Display Bugfix
|
||||
|
||||
### Issue Identified
|
||||
Images saved by Phase 3's auto-photo-save feature were not being displayed in the UI, even though the backend was correctly saving and returning the photo data.
|
||||
|
||||
### Root Cause
|
||||
**Type system mismatch**: The backend's Item response schema includes `photo_path`, `photo_thumbnail_path`, and `photo_upload_date` fields, but the frontend's Item interface (in `frontend/lib/db.ts`) was missing these fields. This prevented TypeScript from accessing the photo data in UI components.
|
||||
|
||||
### Changes Made (v1.14.1)
|
||||
1. **frontend/lib/db.ts**: Added missing photo fields to Item interface
|
||||
- `photo_path?: string`
|
||||
- `photo_thumbnail_path?: string`
|
||||
- `photo_upload_date?: string`
|
||||
|
||||
2. **frontend/components/ItemDetailModal.tsx**: Updated to use photo fields
|
||||
- Changed initialization to check `photo_path` first, then fallback to `image_url`
|
||||
|
||||
3. **frontend/components/InventoryTable.tsx**: Updated to use photo fields
|
||||
- Changed thumbnail check to use `photo_path` with fallback to `image_url`
|
||||
- Updated PhotoModal trigger to use new fields
|
||||
|
||||
4. **frontend/hooks/useItemCreate.ts**: Fixed TypeScript error
|
||||
- Replaced `toast.warning()` (doesn't exist) with `toast.success()`
|
||||
|
||||
### Testing
|
||||
- Frontend build: ✅ Compiles successfully
|
||||
- No regressions: TypeScript strict mode passes
|
||||
|
||||
### Commits
|
||||
- `b5fb2a8c` - fix: add photo_path fields to frontend Item interface
|
||||
- `eaa2d2d2` - fix: replace toast.warning with toast.success
|
||||
- `a62e4e0b` - build: version bump to v1.14.1
|
||||
|
||||
**Status:** ✅ **FIXED** — Images now display correctly in both ItemDetailModal and InventoryTable
|
||||
|
||||
---
|
||||
|
||||
|
||||
Reference in New Issue
Block a user