전체 대화 탭에서도 사진을 그 단계 증빙으로 넣는다 - #378
Merged
Merged
Conversation
단계별 창에만 「검수 사진으로 등록」이 있었다. 전체 대화 탭에서 사진을 보고도 증빙으로 넣으려면 단계 탭으로 옮겨 그 단계를 찾아 다시 열어야 했다(제보). 글마다 어느 단계 이야기인지 이미 붙어 있으므로 **그 글의 단계**로 넣는다. 입력칸에서 고른 단계는 「다음에 쓸 글」의 것이라 쓰지 않는다 — 그걸로 넣으면 A 단계 사진이 B 단계 증빙이 된다. 시간순으로 섞여 있어 어디로 들어가는지 보이지 않으므로 단계 이름을 버튼에 적는다. 곁다리로 두 가지 - 조회만 하는 역할(영업)에는 버튼을 두지 않는다. 서버가 ADMIN·MAKER 만 받아 눌러 봐야 403 이다. 단계별 창도 같이 맞췄다. - 사진이 그려진 뒤 목록을 한 번 더 내린다. 바닥으로 내리는 것이 글 수가 바뀔 때 한 번뿐이라, 사진이 자리를 차지하면 마지막 글의 아래끝이 밖으로 밀렸다 — 실측 40px, 딱 버튼 높이만큼 보이지 않았다. 위를 읽고 있는 사람은 끌어내리지 않는다. 브라우저에서 눌러 확인했다. 주문 #19 「차량 도착」 대화 사진 → `19.홍길동_차량도착_2.png` 로 그 단계 증빙에 들어갔다. 검사 5개. 넷을 되돌려 각각 걸리는 것을 확인했다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #377
무엇을 더했나
전체 대화 탭의 사진 말풍선 아래에 「〈단계이름〉 검수 사진으로 등록」 버튼을 뒀다. 단계별 창과 같은 자리·같은 동작이다.
넣는 곳은 그 글이 달린 단계다. 입력칸에서 고른 단계는 「다음에 쓸 글」의 것이라 쓰지 않는다 — 그걸로 넣으면 A 단계 사진이 B 단계 증빙으로 들어간다. 시간순으로 섞여 있어 어디로 들어가는지 보이지 않으므로 단계 이름을 버튼에 적는다.
백엔드는 그대로다(
POST /orders/:id/steps/:code/files/from-chat).곁다리로 고친 두 가지
rbac('ADMIN','MAKER')라 영업은 눌러 봐야 403 이다. 단계별 창도 같이 맞췄다(거기도 무조건 보이고 있었다).검증
브라우저에서 실제로 눌렀다. 주문 #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개 통과 · 프론트 클린 빌드 통과. 스키마 변경 없음.