통합 문서함/근로자 서류 실제 Document API 연동 - #175
Merged
Merged
Conversation
DocumentListPage/DocumentDetailPage/WorkerDetailPage 서류 섹션이 demo 데이터 대신 fowoco/server 통합 문서함 API를 쓰도록 교체 (server#57). - DocumentListPage: GET /documents, 검색 파라미터가 없어 클라이언트 필터링, 탭 카운트도 받아온 페이지 기준으로 계산 - DocumentDetailPage: 단건 조회 API가 없어 전체 목록에서 worker_document_id로 탐색. 근로자 링크는 실제 worker_id로 연결 (기존 이름 매칭 제거) - WorkerDetailPage 서류 섹션: GET /documents?workerId=로 실제 연동 - 이제 완전히 안 쓰는 workerListData.ts/documentListData.ts 삭제 DocumentItemResponse에 version 필드가 없어 PATCH(확인완료/반려)는 안전하게 호출할 수 없어 화면 상태로만 남김. POST /files는 image/pdf만 허용해서 HWP/HWPX 업로드 데모(#158)는 그대로 유지. Closes #174
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
1 participant
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.
뭘 했나
서버 저장소 -> 통합 문서함·파일 업로드 API가 새로 병합 하신거 확인(fowoco/server#57). 그거에 맞춰서:
GET /documents로 실제 목록 붙임. 검색 파라미터가 서버에 없어서 클라이언트에서 필터링, 탭 개수도 받아온 데이터 기준으로 계산worker_document_id로 찾음. 근로자 연결 링크도 이제 진짜worker_id로 감 (예전엔 이름 문자열 매칭이었음)GET /documents?workerId=로 실제 연동workerListData.ts/documentListData.ts삭제함발견한 것들
DocumentItemResponse에version필드가 없어서,PATCH /workers/{workerId}/documents/{id}(확인완료/반려)에 필요한expected_version을 알 방법이 없음. 그래서DocumentDetailPage의 확인/반려 버튼은 여전히 화면에서만 반영되는 데모임 — 서버에version추가되면 그때 마저 연동하면 됨POST /files는 image/jpeg·png·webp·application/pdf만 허용해서, #158에서 만든 HWP/HWPX 업로드 데모는 이 API로 못 바꿈 (여전히 데모)document-readiness/document-request-draft두 API는 아직 안 건드림 — CaseDetailPage 완료조건/문서탭에 붙이면 좋을 것 같아서 이슈에 후속 후보로 남겨둠테스트
npm run lintnpm run test -- --run(284개 통과)npm run buildCloses #174