차량만 견적의 계약서 생성 차단 · 메일은 견적서만 - #300
Merged
Merged
Conversation
차량만 견적은 계약서를 만들지 않기로 정해 놓고 **화면에는 그대로 열려 있었다.** 지금 계약서는 「특장 매매 및 구조변경 계약서」라, 특장을 장착하지 않는 거래에 쓰면 특장 조항과 금액칸이 공란인 계약서가 고객에게 나간다. **버튼은 자리를 지키되 회색으로 잠근다** — 계약서 생성 · 서명 요청 · 서명본 등록. 없애면 「계약서가 어디 갔지」가 되고, 회색으로 두면 눌러 보고 이유를 알 수 있다. **서버도 막는다.** 화면만 막으면 API 로 그대로 만들어진다. 계약서 렌더·전자서명 발송· 서면계약 등록 **세 길이 모두** `assertContractable` 을 지난다. 고객에게 나가는 문서라 두 겹으로 막는다. **메일 전달은 막지 않는다** — 계약서만 없는 것이지 견적서가 없는 것이 아니다. 대신 팝업의 「계약서도 함께 첨부」 체크칸을 **아예 띄우지 않는다.** 체크할 수 없는 칸을 회색으로 남기면 「왜 안 되지」를 묻게 되는데, 답은 「원래 없는 서류」다. 감추기만 하고 실제로는 첨부되는 일이 없도록 보내는 값도 함께 잠갔다. Closes #299
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 #299
정해 놓고 구현하지 않았던 것을 채운다. 주석에는 「계약서를 만들지 않는다」고 적혀 있었는데 화면 버튼은 그대로 열려 있었다.
잠그는 것 — 자리는 지킨다
없애지 않고 회색으로 남긴다 — 사라지면 「계약서가 어디 갔지」가 되고, 남아 있으면 눌러 보고 이유를 알 수 있다.
서버도 막는다
화면만 막으면 API 로 그대로 만들어진다. 계약서는 고객에게 나가는 문서라 두 겹으로 막았다.
세 길이 모두
assertContractable을 지난다:renderContractPdfForQuote)sendContract)registerPaperContract)메일 전달은 살린다
계약서만 없는 것이지 견적서가 없는 것이 아니다. 버튼은 그대로 두고, 팝업의 「계약서도 함께 첨부」 체크칸을 아예 띄우지 않는다.
체크할 수 없는 칸을 회색으로 남기면 「왜 안 되지」를 묻게 되는데, 답은 「원래 없는 서류」다. 그럴 바엔 칸 자체가 없는 편이 낫다.
감추기만 하고 실제로는 첨부되는 일이 없도록 보내는 값도 함께 잠갔다(
include_contract: attachContract).회귀 가드
셋을 되살려 실제로 깨지는 것까지 확인했다:
마지막 것이 특히 중요하다 — 화면과 실제 동작이 갈리면 아무도 모른 채 계약서가 나간다.
기존 실패 9건은 이 브랜치와 무관.