Skip to content

feat: 경력증명서 PDF 레이아웃을 새 서식에 맞춤 - #78

Merged
tlgms merged 2 commits into
developfrom
31-경력증명서-개별-등록post-apicertificatescreate-미구현으로-405-응답
Oct 6, 2026

Hidden character warning

The head ref may contain hidden characters: "31-\uacbd\ub825\uc99d\uba85\uc11c-\uac1c\ubcc4-\ub4f1\ub85dpost-apicertificatescreate-\ubbf8\uad6c\ud604\uc73c\ub85c-405-\uc751\ub2f5"
Merged

tlgms merged 2 commits into
developfrom
31-경력증명서-개별-등록post-apicertificatescreate-미구현으로-405-응답

Conversation

@tlgms

@tlgms tlgms commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • 경력증명서 PDF 출력 레이아웃을 새 「경력증명서 서식」에 맞게 변경했습니다.

Related Issue

  • 없음

Current vs Improved

  • 변경 전: 재직사항 표에 퇴직사유 열이 행마다 있음, 성명(한글/영문) 가로 배치·생년월일 별도 행, 제목이 문서번호 아래, "위와 같이 경력을 증명합니다." 가운데 정렬
  • 변경 후:
    • 제목(밑줄)을 맨 위로, 그 아래 제 … 호 / 담당자·연락처
    • 성명 칸에 (한글)/(영문) 세로 배치, 같은 행에 생년월일
    • 재직사항 열을 부터·까지·근무부서·담당업무 4개로 정리
    • 총 근무기간 행에 퇴직사유 칸 추가 — 칸이 하나라 마지막(가장 최근 입사) 이력의 사유만 출력
    • 문구를 "위와 같이 재직ㆍ경력을 증명합니다."로 바꾸고 왼쪽 정렬, 발급일 가운데, 발급자 오른쪽
    • 라벨 칸 회색 배경 제거, 빈 행 높이 상향
  • docs/certificate-domain.md 서식 매핑 표의 퇴직사유 항목 갱신

Impact

  • 발급·미리보기 PDF가 실제 서식과 같은 모양으로 출력됩니다. API·DB 변경 없음.

Testing

  • CertificatePdfRendererTest, CertificateIssueServiceTest 통과 (재직사항 10행을 채워도 1페이지 유지 포함)
  • 샘플 PDF를 렌더해 서식 PDF와 육안 비교

Risk

  • 이력이 여러 건이면 이전 이력들의 퇴직사유는 증명서에 출력되지 않습니다.

Checklist

  • 의도하지 않은 기능적 동작 변경 없음
  • 해당 시 메트릭 / 벤치마크 결과 업데이트 완료

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Improvements
    • Updated the certificate layout with clearer sections for personal details, work history, total work period, retirement reasons, other information, and purpose.
    • Work history now has space for at least 10 entries, and retirement reasons are shown together in a separate summary rather than repeated alongside each entry.
    • Refined the certificate’s headings, spacing, and issue-date and issuer presentation.

- 성명 한글/영문 세로 배치, 생년월일을 성명 행으로 이동
- 재직사항에서 퇴직사유 열 제거, 총 근무기간 행에 퇴직사유 칸 추가 (사유는 중복 제거 후 이어 붙임)
- 제목 위치·밑줄, 증명 문구("재직ㆍ경력") 및 하단 정렬 변경

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9f4d0d4b-5058-44c1-9c60-7abb91d62c19
📥 Commits

Reviewing files that changed from the base of the PR and between ca55da6 and 1bdfb17.

📒 Files selected for processing (2)
  • docs/certificate-domain.md
  • src/main/java/commonly/commonlybe/certificate/document/CertificatePdfRenderer.java
 ___________________________________
< I like big bugs and I cannot lie. >
 -----------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cowtree28
cowtree28 self-requested a review October 6, 2026 05:15
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.

2 participants