Skip to content

bug: 모집 정원 판정 기준 통일 (fix/recruit-status-capacity) - #437

Merged
kimjiy9607 merged 2 commits into
betafrom
fix/recruit-status-capacity
Nov 8, 2026
Merged

kimjiy9607 merged 2 commits into
betafrom
fix/recruit-status-capacity

Conversation

@kimjiy9607

Copy link
Copy Markdown
Contributor

요약

작업 요약

버그 수정: 모집 정원 판정 기준 통일 (fix/recruit-status-capacity)

문제: 모집 중/마감 판정이 화면마다 달랐다. 하차(WITHDRAWN) 후에도 신청 행이 남아 있어 정원이 영구 마감으로 고정되는 버그.

원인: 목록·상세·백오피스·SQL 필터가 각각 다른 기준(신청 건수 vs 명부 인원)으로 정원을 계산.

결정 D4-A: 정원을 차지하는 인원 = 명부 ACTIVE + CAPACITY_ROLES(MEMBER·LEADER). 하차(WITHDRAWN) 시 자리 반납.

변경 사항

체크리스트

Self Review

  • 내 코드를 직접 리뷰했다 (/self-review 실행 또는 수동)
  • 불필요한 console.log / println / 디버그 코드를 제거했다
  • 변수·함수 이름이 의도를 명확히 드러낸다
  • 에러 핸들링이 빠진 경로가 없다

테스트

  • Stage 에서 테스트했다 (해당 시 스크린샷 첨부)
  • 단위/통합 테스트를 추가 또는 업데이트했다
  • 기존 테스트가 깨지지 않는다 (./gradlew test / pnpm test)

보안 (PUBLIC 레포)

  • 시크릿·토큰·API 키·PII 가 코드에 없다
  • .env 파일이 커밋에 포함되지 않았다

기타

  • 스키마 변경 시 V{n}__*.sql 마이그레이션을 같이 넣었다
  • 관련 문서(ERD, 스펙)를 업데이트했다
  • 01-planning/stories/** 또는 frontend/apps/playground/**/spec.ts 를 건드렸다면 pnpm --filter playground export-specs --check 통과 확인

팀 공유

  • 팀이 알아야 할 결정이 생겼다 → docs/share/ 에 한 장 (없으면 지운다)
  • prod 사고 수정이다 → docs/incidents/ 에 한 장 (없으면 지운다)

Jiyoon Kim and others added 2 commits November 7, 2026 18:20
하차(WITHDRAWN) 후 신청 행이 남아 정원이 영구 마감되는 버그 수정.
목록·상세·백오피스·SQL 필터 모두 STUDY_PARTICIPANT ACTIVE + CAPACITY_ROLES 로 통일.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…dyListIntegrationTest FK

BackofficeStudyListService: programs 맵 미존재 시 NPE → Optional.ofNullable + orElseThrow 로 보호
AdminStudyListIntegrationTest: insertParticipant STUDY_GROUP_ID=0 FK 위반 → STUDY_GROUP_ID=9611L 추가

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kimjiy9607
kimjiy9607 requested a review from titaniper as a code owner November 8, 2026 02:28
@kimjiy9607
kimjiy9607 merged commit 0fd4d99 into beta Nov 8, 2026
3 checks passed
@kimjiy9607
kimjiy9607 deleted the fix/recruit-status-capacity branch November 8, 2026 02:31
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