Task #64
closedTask #52: Coding
Coding-Facility Register/Edit
100%
Files
Updated by Anonymous 5 months ago
- % Done changed from 70 to 100
Hi Nghĩa, task này tui vừa tạo PR check giúp tui nhen
https://github.com/clmt-vn/facility-owner/pull/4
Updated by Nghia Nguyen 5 months ago
- Assignee changed from Nghia Nguyen to Anonymous
Hello @duy, tui có note mấy ý sau
-
Resolve conflict
-
施設ID→ Cần phải disable khi edit và ẩn khi register.施設IDko cần đánh dấu required field vì nó ko thể chỉnh sửa. Trong BD施設IDlà short text box field. -
Field
駐車場備考sau khi lưu ko hiển thị ra page detail. Duy điều tra xem thử lỗi từ page detail hay do page edit ko lưu đc nha. Nếu lỗi từ page detail thì cũng fix luôn nha vì nhánh detail đã merge rồi. -
Nếu disable Facility (status = 0) → disable tất cả Place của Facility đó→ chưa thấy xử lí -
Tại sao hardcode đường dẫn "app/owner/facility/1000/tmp_"? (Tui có comment dòng trong github) → Chức năng upload ảnh của facility trước đó thực hiện bình thường, ko có lỗi liên quan đến đường dẫn → ko nên sửa hàm file path này. Nếu lỗi liên quan đến đường dẫn xảy ra ở local thì cần xem lại config hoặc cấp quyền.
-
Chức năng copy đang có lỗi. Thực hiện bước: Nhấn vào xem detail facility → chọn コピーして新規作成 (Copy) → Chọn 登録 (Lưu) → Xảy ra lỗi nhưng ko thông báo message nên ko biết tại sao bị lỗi. Bên staging hiện tại có thông báo nguyên nhân lỗi
Updated by Anonymous 5 months ago
Tui mới fix với update follow theo comment, có gì ông check lại giúp tui nhen
https://github.com/clmt-vn/facility-owner/pull/4
Updated by Nghia Nguyen 5 months ago
Hi @duy ơi, lúc tạo mà bị lỗi thì tui nghĩ ko nên show error từ phía server lên màn hình thế này. Ông đừng quan tâm cái error trong trong hình nha, này do table chưa update nên bị lỗi thôi. Nhưng ông xem có cách nào đừng show error từ phía server lên đây nha. Mấy kiểu này nên là internal error thôi

Updated by Nghia Nguyen 5 months ago
có mấy chỗ liên quan tới xử lí file được upload tui có comment bên PR mà chắc Duy bị miss. Duy xem giúp tui nha.
File bên chỗ facilities bây giờ mình chỉ lưu 5 loại này thôi, mấy loại khác mình ko xử lí nữa
写真TOP(外観)
搬入口
フロアマップ
その他1
その他2
Updated by Anonymous 5 months ago
Hi Nghĩa
- đoạn dư field tui có update rồi
- bên chỗ log lỗi tui cũng vừa add try catch transaction, ko để nó log lỗi từ dưới db lên, check giúp tui luôn nhen
https://github.com/clmt-vn/facility-owner/pull/4
Updated by Nghia Nguyen 5 months ago
- Assignee changed from Nghia Nguyen to dang dao