docs: CTO report — both bugs verified fixed on Chrome UI

P2: POS duplicates were DB seed issue (9 records deleted), now 9 unique products.
P3: Settings shop name now shows "Cobic Coffee" correctly.
Both verified visually on Chrome.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Ho Ngoc Hai
2026-03-13 21:31:09 +07:00
parent db72640198
commit dd74b05189

View File

@@ -59,17 +59,19 @@ Toàn bộ tính năng Admin và Staff POS đã được kiểm tra trực tiế
## Bugs Found & Fixed (commit 344be33)
### P2: Duplicate Products in POS Grid — ✅ FIXED
### P2: Duplicate Products in POS Grid — ✅ FIXED & VERIFIED
- **URL**: /pos/{shopId}/cafe
- **Root Cause**: BFF CatalogController không filter `isActive`, trả về cả active + inactive products
- **Fix**: BFF default `isActive=true` + client-side dedup by product ID in PosDataService
- **Files**: CatalogController.cs, PosDataService.cs
- **Root Cause**: Seed data chạy 2 lần → 18 products với IDs khác nhau nhưng cùng name/price trong catalog DB
- **Fix**: (1) Xóa 9 duplicate records trong DB, (2) BFF default `isActive=true`, (3) client-side dedup by ID as safety net
- **Files**: CatalogController.cs, PosDataService.cs + DB cleanup
- **Verified**: POS giờ hiển thị đúng 9 sản phẩm unique
### P3: Admin Settings — Tên cửa hàng hiện "--" — ✅ FIXED
### P3: Admin Settings — Tên cửa hàng hiện "--" — ✅ FIXED & VERIFIED
- **URL**: /admin/shop/{shopId}/settings
- **Root Cause**: ShopPage.razor không truyền ShopName/VerticalLabel parameters cho ShopSettings component
- **Fix**: Pass `ShopName="@_shopName" VerticalLabel="@_verticalLabel"` to ShopSettings
- **Files**: ShopPage.razor
- **Verified**: Settings hiện "Cobic Coffee" / "Vertical_Cafe" đúng
---