Skip to content

차량만 견적의 계약서 생성 차단 · 메일은 견적서만 - #300

Merged
Jay722J merged 1 commit into
mainfrom
fix/vehicle-only-no-contract
Aug 26, 2026
Merged

차량만 견적의 계약서 생성 차단 · 메일은 견적서만#300
Jay722J merged 1 commit into
mainfrom
fix/vehicle-only-no-contract

Conversation

@Jay722J

@Jay722J Jay722J commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Closes #299

정해 놓고 구현하지 않았던 것을 채운다. 주석에는 「계약서를 만들지 않는다」고 적혀 있었는데 화면 버튼은 그대로 열려 있었다.

잠그는 것 — 자리는 지킨다

버튼
계약서 생성 회색 · 잠금
서명 요청 회색 · 잠금
서명본 등록(종이) 회색 · 잠금

없애지 않고 회색으로 남긴다 — 사라지면 「계약서가 어디 갔지」가 되고, 남아 있으면 눌러 보고 이유를 알 수 있다.

서버도 막는다

화면만 막으면 API 로 그대로 만들어진다. 계약서는 고객에게 나가는 문서라 두 겹으로 막았다.

세 길이 모두 assertContractable 을 지난다:

  • 계약서 렌더 (renderContractPdfForQuote)
  • 전자서명 발송 (sendContract)
  • 서면계약 등록 (registerPaperContract)

메일 전달은 살린다

계약서만 없는 것이지 견적서가 없는 것이 아니다. 버튼은 그대로 두고, 팝업의 「계약서도 함께 첨부」 체크칸을 아예 띄우지 않는다.

체크할 수 없는 칸을 회색으로 남기면 「왜 안 되지」를 묻게 되는데, 답은 「원래 없는 서류」다. 그럴 바엔 칸 자체가 없는 편이 낫다.

감추기만 하고 실제로는 첨부되는 일이 없도록 보내는 값도 함께 잠갔다(include_contract: attachContract).

회귀 가드

셋을 되살려 실제로 깨지는 것까지 확인했다:

되살린 버그 잡히는가
서버 가드 제거
계약 버튼 잠금 제거
첨부칸만 감추고 실제로는 계약서 전송

마지막 것이 특히 중요하다 — 화면과 실제 동작이 갈리면 아무도 모른 채 계약서가 나간다.

기존 실패 9건은 이 브랜치와 무관.

차량만 견적은 계약서를 만들지 않기로 정해 놓고 **화면에는 그대로 열려 있었다.**
지금 계약서는 「특장 매매 및 구조변경 계약서」라, 특장을 장착하지 않는 거래에 쓰면
특장 조항과 금액칸이 공란인 계약서가 고객에게 나간다.

**버튼은 자리를 지키되 회색으로 잠근다** — 계약서 생성 · 서명 요청 · 서명본 등록.
없애면 「계약서가 어디 갔지」가 되고, 회색으로 두면 눌러 보고 이유를 알 수 있다.

**서버도 막는다.** 화면만 막으면 API 로 그대로 만들어진다. 계약서 렌더·전자서명 발송·
서면계약 등록 **세 길이 모두** `assertContractable` 을 지난다. 고객에게 나가는 문서라
두 겹으로 막는다.

**메일 전달은 막지 않는다** — 계약서만 없는 것이지 견적서가 없는 것이 아니다.
대신 팝업의 「계약서도 함께 첨부」 체크칸을 **아예 띄우지 않는다.** 체크할 수 없는 칸을
회색으로 남기면 「왜 안 되지」를 묻게 되는데, 답은 「원래 없는 서류」다.
감추기만 하고 실제로는 첨부되는 일이 없도록 보내는 값도 함께 잠갔다.

Closes #299
@Jay722J
Jay722J merged commit c5fbee4 into main Aug 26, 2026
@Jay722J
Jay722J deleted the fix/vehicle-only-no-contract branch August 26, 2026 03:28
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