Skip to content

feat(pnl): 「입력 필요」는 계약 체결부터 — 전자서명 완료 또는 서면계약 스캔본 등록 - #476

Merged
Jay722J merged 1 commit into
mainfrom
feat/pnl-pending-on-contract
Sep 17, 2026
Merged

Jay722J merged 1 commit into
mainfrom
feat/pnl-pending-on-contract

Conversation

@Jay722J

@Jay722J Jay722J commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Closes 475

기준 변경

전 후
「입력 필요」에 서는 때 특장사 수락 계약 체결(전자서명 완료 · 서면계약 스캔본 등록)
순서 수락일 오래된 순 체결일 오래된 순
목록 날짜 수락 YYYY-MM-DD 계약 YYYY-MM-DD

무엇으로 보나

두 길 모두 계약 줄(purchase_contract)을 COMPLETED 로 남기고 체결 시각(completed_at)을 적는다 —
그 줄 하나로 두 길을 함께 본다.

⚠️ 견적 상태(계약완료 이상)로 고르지 않는다. 상태는 배정 취소·거부·주문 삭제로 오가지만
체결됐다는 사실은 계약 줄에만 정확히 남는다. 서명 요청만 보낸 건(SENT)은 체결이 아니다.

⚠️ 배포 뒤 운영에서 보일 것

손익 탭 이전에 체결된 계약도 발행일이 안 적혀 있으면 모두 「입력 필요」에 선다.
처음 열면 목록이 꽤 길 수 있다(체결일 오래된 순이라 맨 위부터 처리하면 된다).

DB

변경 없음.

검증

  • 백엔드 931개 · shared 351개 전부 통과
    • 전자서명 완료면 선다 · 서명 요청만 보낸 건은 아니다 · 계약 줄 없이 상태만 계약완료인 건도 아니다
    • 서면계약 스캔본을 올려도 선다 · 특장사 수락 여부와 상관없다 · 체결일 오래된 순
  • 로컬: 실제 체결 2건 + 더미 2건이 체결일 순(08-18, 08-18, 09-15, 09-16)으로 뜨는 것 확인

🤖 Generated with Claude Code

지시로 기준을 바꾼다. 그 전에는 **특장사 수락** 시점이었다. 계약만 끝나도 세금계산서 준비는
시작되므로 앞당긴다.

## 무엇으로 보나

두 길(전자서명 완료 · 서면계약 스캔본 등록) 모두 계약 줄을 `COMPLETED` 로 남기고 체결 시각
(`completed_at`)을 적는다 — **그 줄 하나로 두 길을 함께 본다.**

⚠️ 견적 상태(계약완료 이상)로 고르지 않는다. 상태는 배정 취소·거부·주문 삭제로 오가지만,
   **체결됐다는 사실**은 계약 줄에만 정확히 남는다. 서명 요청만 보낸 건(SENT)은 체결이 아니다.

## 순서

체결일이 오래된 것이 맨 위다(재계약이면 가장 이른 체결일). 관계 칸으로는 DB 가 정렬하지 못해
받아 온 뒤 줄 세운다. 목록의 날짜 표시도 「수락 …」 → 「계약 …」.

## 검사

 · 전자서명이 끝나면 선다 · 서명 요청만 보낸 건은 아니다 · 계약 줄 없이 상태만 계약완료인 건도 아니다
 · 서면계약 스캔본을 올려도 선다(체결일이 실린다)
 · 특장사 수락 여부와 상관없다(주문이 아예 없어도 선다)
 · 체결일이 오래된 것이 맨 위다

로컬 더미 스크립트도 체결된 계약 줄을 만들도록 고쳤다(전자서명·서면계약을 섞어서).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@Jay722J
Jay722J merged commit 3999790 into main Sep 17, 2026
4 checks passed
@Jay722J
Jay722J deleted the feat/pnl-pending-on-contract branch September 17, 2026 04:21
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