Skip to content

update(application): 학생 외부 지원 등록 시 면접 예정 상태로 초기화 - #87

Merged
hongjm0912 merged 2 commits into
developfrom
fix/student-external-application-interview-status
Aug 30, 2026
Merged

hongjm0912 merged 2 commits into
developfrom
fix/student-external-application-interview-status

Conversation

@hongjm0912

@hongjm0912 hongjm0912 commented Aug 30, 2026 •

Copy link
Copy Markdown
Contributor

배경

준연님이 학생 외부 지원 내역 수동 등록 기능(POST /api/applications/external, PR #82)을 검토하고 수정이 필요한 부분을 정리해주셨습니다.

  1. 별도의 "지원 일시" 입력/필드는 필요 없음
  2. 외부 지원 내역을 등록하면 상태가 "결과 대기"가 아닌 "면접 예정"으로 설정되어야 함
  3. 학생이 외부 지원 내역을 직접 등록하는 상황은 이미 서류에 합격해 면접 단계에 들어간 경우를 전제로 하므로, 등록 시 면접 일시도 함께 입력받아 저장해야 함

즉, 이 API의 성격을 "외부 지원 접수 기록"에서 "이미 서류 합격 후 면접이 잡힌 외부 지원 건 기록"으로 명확히 하는 변경입니다. 대화 중 "지원 일시(appliedAt)"가 정말 필요 없는 필드로 확인되어, 별도 입력을 안 받는 데서 더 나아가 필드 자체를 제거하기로 했습니다.

변경 사항

  • POST /api/applications/external 요청 바디에 interviewAt(면접 일시) 추가, 필수값으로 검증(없으면 400)
  • RegisterMyExternalApplicationService: 등록 시 상태를 APPLIED가 아닌 INTERVIEW_SCHEDULED로 설정(ApplicationJpaEntity.updateStatus() 재사용 — RegisterManualApplicantService의 기존 패턴과 동일)
  • ApplicationResponse(학생 "내 지원 현황" 응답)에 interviewAt 노출 — 등록 직후 응답 및 이후 조회에서 면접 일시를 확인할 수 있도록 함
  • ApplicationJpaEntity, ApplicationResponse, AdminApplicantResponse에서 지원 일시(appliedAt) 필드 완전 제거
    • 조회/정렬 등 어디에도 사용되지 않는 순수 표시용 필드였음을 확인 후 제거
    • dev DB의 tb_application.applied_at 컬럼도 함께 DROP COLUMN 처리(사용자 확인 후 직접 실행) — ddl-auto: update는 컬럼을 삭제하지 않고, 남겨두면 NOT NULL 제약 때문에 신규 INSERT가 전부 실패하므로 필요한 조치였음

검증

  • ./gradlew compileJava 통과
  • dev DB에서 기존 등록 데이터(application_source='EXTERNAL')가 모두 status=APPLIED로 저장되어 있음을 확인 — 변경 전 동작이 의도(APPLIED 고정)대로였음을 재검증 후 이번 스펙 변경 진행
  • dev DB tb_application.applied_at 컬럼 삭제 및 나머지 컬럼 정상 확인

@hongjm0912 hongjm0912 self-assigned this Aug 30, 2026
@hongjm0912
hongjm0912 merged commit 6187589 into develop Aug 30, 2026
1 check passed
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