Skip to content

전체 대화 탭에서도 사진을 그 단계 증빙으로 넣는다 - #378

Merged
Jay722J merged 1 commit into
mainfrom
feat/chat-tab-promote-evidence
Sep 10, 2026
Merged

전체 대화 탭에서도 사진을 그 단계 증빙으로 넣는다#378
Jay722J merged 1 commit into
mainfrom
feat/chat-tab-promote-evidence

Conversation

@Jay722J

@Jay722J Jay722J commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Closes #377

무엇을 더했나

전체 대화 탭의 사진 말풍선 아래에 「〈단계이름〉 검수 사진으로 등록」 버튼을 뒀다. 단계별 창과 같은 자리·같은 동작이다.

넣는 곳은 그 글이 달린 단계다. 입력칸에서 고른 단계는 「다음에 쓸 글」의 것이라 쓰지 않는다 — 그걸로 넣으면 A 단계 사진이 B 단계 증빙으로 들어간다. 시간순으로 섞여 있어 어디로 들어가는지 보이지 않으므로 단계 이름을 버튼에 적는다.

백엔드는 그대로다(POST /orders/:id/steps/:code/files/from-chat).

곁다리로 고친 두 가지

  • 조회만 하는 역할에는 두지 않는다. 서버가 rbac('ADMIN','MAKER') 라 영업은 눌러 봐야 403 이다. 단계별 창도 같이 맞췄다(거기도 무조건 보이고 있었다).
  • 사진이 그려진 뒤 목록을 한 번 더 내린다. 바닥으로 내리는 것이 글 수가 바뀔 때 한 번뿐이라, 그 뒤에 사진이 자리를 차지하면 마지막 글의 아래끝이 밖으로 밀렸다 — 실측 40px, 딱 버튼 높이만큼 보이지 않았다. 위를 읽고 있는 사람은 끌어내리지 않는다(바닥에서 120px 안쪽일 때만).

검증

브라우저에서 실제로 눌렀다. 주문 #19 「차량 도착」 대화 사진 → 버튼 문구 「차량 도착」 검수 사진으로 등록 → 누른 뒤 ✓ 검수 사진으로 등록됨, DB 에 19.홍길동_차량도착_2.png(inspection_photo, step_code=car_arrived, uploaded_by=demo-maker@local) 로 들어갔다.

버튼 위치는 눈으로 보지 않고 쟀다 — 고치기 전 버튼 [755,777] / 목록 [296,753] → 보이나 false, 고친 뒤 버튼 [715,737] → 보이나 true.

검사 5개(due-nudge.test.ts 의 「대화 사진 → 증빙」). 넷을 되돌려 각각 걸리는 것을 확인했다.

되돌린 것 걸린 검사
입력칸에서 고른 단계로 넣음 넣는 곳은 그 글이 달린 단계다
영업에게도 보이게 함 조회만 하는 역할에는 두지 않는다
사진 로드 뒤 다시 안 내림 사진이 그려진 뒤 목록을 다시 내린다
증빙 등록 자체를 뺌 전체 대화 탭에서도 사진을 증빙으로 넣을 수 있다

백엔드 전체 677개 통과 · 프론트 클린 빌드 통과. 스키마 변경 없음.

단계별 창에만 「검수 사진으로 등록」이 있었다. 전체 대화 탭에서 사진을
보고도 증빙으로 넣으려면 단계 탭으로 옮겨 그 단계를 찾아 다시 열어야
했다(제보).

글마다 어느 단계 이야기인지 이미 붙어 있으므로 **그 글의 단계**로 넣는다.
입력칸에서 고른 단계는 「다음에 쓸 글」의 것이라 쓰지 않는다 — 그걸로
넣으면 A 단계 사진이 B 단계 증빙이 된다. 시간순으로 섞여 있어 어디로
들어가는지 보이지 않으므로 단계 이름을 버튼에 적는다.

곁다리로 두 가지

- 조회만 하는 역할(영업)에는 버튼을 두지 않는다. 서버가 ADMIN·MAKER 만
  받아 눌러 봐야 403 이다. 단계별 창도 같이 맞췄다.
- 사진이 그려진 뒤 목록을 한 번 더 내린다. 바닥으로 내리는 것이 글 수가
  바뀔 때 한 번뿐이라, 사진이 자리를 차지하면 마지막 글의 아래끝이 밖으로
  밀렸다 — 실측 40px, 딱 버튼 높이만큼 보이지 않았다. 위를 읽고 있는
  사람은 끌어내리지 않는다.

브라우저에서 눌러 확인했다. 주문 #19 「차량 도착」 대화 사진 →
`19.홍길동_차량도착_2.png` 로 그 단계 증빙에 들어갔다.

검사 5개. 넷을 되돌려 각각 걸리는 것을 확인했다.
@Jay722J
Jay722J merged commit 4fe9d5d into main Sep 10, 2026
4 checks passed
@Jay722J
Jay722J deleted the feat/chat-tab-promote-evidence branch September 10, 2026 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

전체 대화 탭에서 사진을 증빙으로 등록할 수 없다

1 participant