Facility-Owner

spec
https://columbusmantech-my.sharepoint.com/:x:/r/personal/d_dang_columbusmantech_vn/_layouts/15/Doc.aspx?sourcedoc=%7B22C1F773-BEC2-4FF4-83CD-E9419AE21E87%7D&file=ScreenDesign_FacilityList_v1.0.xlsx&action=default&mobileredirect=true

hi @duy, mình có review như sau:

Feature:

1. 施設ID có max length là 20 nhưng Duy đang để max value là 20 nên nhập số từ 21 trở đi đang bị lỗi.
2. Field 店舗担当者 ko hoạt động, ko query được.
3. Mình chưa thấy xử lí case: Nếu facility có tồn tại ít nhất 1 place thì disable nút xóa.

Coding:

• Nếu đoạn code nào mình ko dùng nữa thì nên remove hẳn thay vì comment (vd: line 122 resources/views/facility/list.blade.php)
• Dropdown 店舗担当者 - ownerList → nên đặt tên là ownerAccountList sẽ rõ nghĩa hơn.
• Remove console.error, ko thông báo error ra browser.
• 店舗担当者 là dạng master data, không thay đổi thường xuyên và chỉ dùng để hiển thị dropdown. Nên fetch từ server-side khi khởi tạo page, ko cần gọi API riêng ở client. Các page khác cũng đang làm theo cách này để đồng bộ.

@duy,

Issue 1 mình thấy chưa được fix. Trong code vẫn để max value là 20 → đẫn đến nhập ID >20 sẽ bị lỗi

Ngoài ra, đang phát sinh lỗi mới khi vừa mới truy page Facility List: Column not found: 1054 Unknown column 'f.place' in 'field list' → Lỗi này mình đoán là đang query field place từ table facilities nhưng thực tế table này ko có field place nên dẫn đến bị lỗi.

@duy kiểm tra lại, test lại r mình review lại nha.

@duy

1. 施設ID có max length là 20 nhưng Duy đang để max value là 20. Chỗ này fix vẫn chưa đúng á, max length là 20 nhưng Duy fix là set max value là 21, chú ý field đó nó đang là numberic nên ko thể set max như v, nó đang hiểu là max value thay vì max length (chỉ có hiệu lực với string)
2. Field 店舗担当者 ko hoạt động, ko query được: Vẫn chưa query đúng á Duy. Chọn 店舗担当者 bất kì rồi bấm search nhưng List trả về ko đúng á.
3. Tui có comment trong source. Nếu query cả danh sách places chỉ để check hasPlace thì chưa tối ưu. Thay vào đó có thêm thêm query để đánh dấu facility has place ( ví dụ: SELECT COUNT(*) FROM places p WHERE place.facility_id = facility.id...)

Còn lại OK r nha

Update lại xíu bên chỗ query nữa là đc nha

review lại cho Duy cho xong đi hén

đã merge

@Nghia Nguyen deploy lên môi trường dev xong assign lại cho a hén

https://facility-owner.columbusmantech.dev

user: mo1@mail.com
pass: 12345678