Project

General

Profile

Actions

Task #101

closed

Task #52: Coding

Coding-PlaceNewEdit

Added by dang dao 5 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Start date:
08/20/2025
Due date:
08/22/2025
% Done:

100%

Estimated time:
(Total: 0:00 h)


Files


Subtasks 1 (0 open1 closed)

Bug #187: Create/Edit Place - Load danh sách Facilities theo organizationClosedY Pham08/20/202508/22/2025

Actions
Actions #1

Updated by Anonymous 5 months ago

  • Due date set to 07/26/2025
Actions #2

Updated by Anonymous 5 months ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 10
Actions #3

Updated by Anonymous 5 months ago

  • Due date changed from 07/26/2025 to 07/28/2025
Actions #4

Updated by dang dao 5 months ago

  • Due date changed from 07/28/2025 to 07/30/2025
Actions #5

Updated by dang dao 5 months ago

  • Due date changed from 07/30/2025 to 07/31/2025
Actions #6

Updated by Anonymous 5 months ago

  • % Done changed from 10 to 100
  • Assignee changed from Anonymous to Nghia Nguyen

Hi Nghĩa tui có update phần facility new/edit ở PR

https://github.com/clmt-vn/facility-owner/pull/12

Có gì ông check giúp tui với hen

Actions #7

Updated by Nghia Nguyen 5 months ago

  • File Screenshot 2025-08-01 at 11.00.07 PM.png added


chọn Facility rồi mà vẫn báo lỗi nè @duy

Actions #8

Updated by Nghia Nguyen 5 months ago

  • File Screenshot 2025-08-01 at 11.01.08 PM.png added

vào trang edit đang bị lỗi

Actions #9

Updated by Nghia Nguyen 5 months ago

  • File Screenshot 2025-08-01 at 11.16.49 PM.png added

Actions #10

Updated by Nghia Nguyen 5 months ago

tui chưa test đc khả năng thêm place. Nhưng đọc code trong BasePlaceSaveRequest thì ko biết Duy có handle việc upload file chưa á

Actions #11

Updated by Nghia Nguyen 5 months ago

  • Assignee changed from Nghia Nguyen to Anonymous
Actions #12

Updated by Anonymous 4 months ago

  • Assignee changed from Anonymous to Nghia Nguyen

Hi Nghĩa chỗ này tui vừa update lại, ông check lại giúp tui nhe ông
https://github.com/clmt-vn/facility-owner/pull/12

Actions #13

Updated by Nghia Nguyen 4 months ago

  • File deleted (Screenshot 2025-08-01 at 11.00.07 PM.png)
Actions #14

Updated by Nghia Nguyen 4 months ago

  • File deleted (Screenshot 2025-08-01 at 11.01.08 PM.png)

Updated by Nghia Nguyen 4 months ago

hi @duy, tui xem qua thì có mấy comment sau:

UI

  1. Update UI field 実施場所ID khi bị disable.
    Hiện tại trông khó phân biệt disabled field

    Theo trong tài liệu, field 実施場所ID khi disable có màu xám.

  1. Update field 施設: Dùng selectbox dài.
    Facility name có 64 ký tự. Selectbox ngắn hiện tại ko thể hiển thị hết.
    Dùng selectbox dài giống trong tài liệu:

  1. 実施場所広さ ko phải require field → Update UI

  1. Edit place ko thấy hiển thị file ảnh đã upload.
    Vd: Tạo place mới, upload 実施場所1 → Lưu → Edit place vừa được tạo → thấy tên file đã upload hiển thị ở 実施場所1

  1. Lỗi khi chọn status là 無効

  1. Lỗi khi truy cập trang copy place

Logic

  1. Place name không được trùng trong cùng một Facility.

  2. File lưu chưa đúng thư mục.
    File được cấu trúc thư mục:
    {PLACE_STOREFILE_PATH}/{place_dir}/{place_id}/

    • picture/ - Ảnh
    • document/ - Tài liệu
      Ví dụ: Nếu place_id = 1234 thì place_dir = 2000
      PLACE_STOREFILE_PATH = /owner/place
      tên ảnh là abc123
      Đường dẫn đầy đủ của ảnh sẽ là: /owner/place/2000/1234/picture/abc123
  3. created_user là user đang được xác thực để tạo place. (đã comment trong PR)

  4. Ko đc hardcode is_test = 1 (đã comment trong PR)

  5. Ko cần get all facilities sau khi tạo place thành công (đã comment trong PR)

Actions #16

Updated by Nghia Nguyen 4 months ago

  • File deleted (Screenshot 2025-08-01 at 11.16.49 PM.png)
Actions #17

Updated by Nghia Nguyen 4 months ago

  1. Danh sách facilities lấy chưa đúng. Chỉ lấy danh sách facilities của organization của user đang đăng nhập.
Actions #18

Updated by Anonymous 4 months ago

  • Assignee changed from Anonymous to Nghia Nguyen

Có gì ông check giúp tui nh Nghĩa
https://github.com/clmt-vn/facility-owner/pull/12

Updated by Nghia Nguyen 4 months ago

hi @duy ơi,

  1. Trên UI 実施場所広さ vẫn đang là required field nè. Nó là optional nha, bỏ 必須 bên phải field.

  2. Error message lúc nhập place trùng chưa đúng. Tui thấy server có trả về error message là "同じ施設内で「test」という実施場所名は既に使用されています。" rồi nhưng status code đang là 500 nên nó hiển thị システムエラーが発生しました。

Duy sửa cái status code là 409 nha. Status code 409 thì sẽ hiển thị message từ server trả về

  1. Viết hàm lấy danh sách facilities (theo organization) để tái sử dụng ở nhiều nơi nha (đã comment trong PR)

  2. Lỗi khi chọn status là 無効 → Lỗi này vẫn bị nha Duy. Khi create hoặc edit place, chọn status (状態) là invalid (無効) thì nó báo lỗi Please select a status(状態を選択してください。). Duy xem lại chỗ này nha

  3. Lỗi khi save.

  • Chọn edit place đã tạo.
  • Ko chỉnh sửa gì → Nhấn lưu (更新)
  • Hiển thị lỗi như bên dưới, mặc dù nó ko phải là required field
  1. Edit place ko thấy hiển thị file ảnh đã upload. → lỗi này vẫn còn.
Actions #20

Updated by Anonymous 4 months ago

Tui update lại rồi ông check giúp tui lại nh
https://github.com/clmt-vn/facility-owner/pull/12

Actions #21

Updated by Anonymous 4 months ago

  • Assignee changed from Anonymous to Nghia Nguyen
Actions #22

Updated by Nghia Nguyen 4 months ago

  • File Screen Recording 2025-08-08 at 10.51.39 PM.mov added
Actions #23

Updated by Nghia Nguyen 4 months ago

  • File deleted (Screen Recording 2025-08-08 at 10.51.39 PM.mov)
Actions #24

Updated by Nghia Nguyen 4 months ago

hi @duy

ông coi lại lỗi này nha, tui thấy vẫn còn nè.
https://www.loom.com/share/ce11b19a1eba4d2e8d0c747d4badd9ec?sid=9e0e2a77-b401-47f9-8715-d1283b71ff2b

cái nào ko tái hiện được thì nhắn tui chứ đừng bỏ qua nha.

Actions #25

Updated by Anonymous 4 months ago

  • Assignee changed from Anonymous to Nghia Nguyen

Hi @Nghia Nguyen
Check lại giúp tui há Nghĩa ơi, vấn đề nằm ở chỗ check filetype, hình như do swift nhánh nhiều quá tui bị mất code á

Actions #26

Updated by Nghia Nguyen 4 months ago

  • Assignee changed from Nghia Nguyen to Anonymous
Actions #27

Updated by Nghia Nguyen 4 months ago

ông fix xong ông test lại full flow giúp tui nha @duy

Actions #28

Updated by Nghia Nguyen 4 months ago

@duy tui merge nhánh place detail vô r. Bây giờ ông pull nhánh develop về nhánh hiện tại để có code mới nhất rồi fix tiếp nha

Actions #29

Updated by Nghia Nguyen 4 months ago

@duy
page place list ko thấy hiển thị facility name ở cột 施設ID/施設名

Actions #30

Updated by Nghia Nguyen 4 months ago

@duy xem lại bên Order Detail giúp t nha

Actions #31

Updated by dang dao 4 months ago

  • Due date changed from 07/31/2025 to 08/12/2025
Actions #32

Updated by Anonymous 4 months ago

Hi @Nghia Nguyen https://github.com/clmt-vn/facility-owner/pull/12 tui có fix lại các issue bên detail với new/edit check lại giúp tui nhen ông

Actions #33

Updated by Anonymous 4 months ago

  • Assignee changed from Anonymous to Nghia Nguyen
Actions #34

Updated by Nghia Nguyen 4 months ago

  • Status changed from In Progress to Resolved
Actions #35

Updated by Nghia Nguyen 4 months ago

Đã merge vào nhánh develop

Actions #36

Updated by Nghia Nguyen 4 months ago

  • Assignee changed from Nghia Nguyen to dang dao

Đã deploy server dev

Actions #37

Updated by Nghia Nguyen 4 months ago

  • Subtask #187 added
Actions #38

Updated by dang dao 3 months ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF