배경
PR #121 리뷰(hoTan35)에서 지적된 항목 — early-start 관련 코드는 acceptApplication 버그픽스(이슈 #120)와 별개 기능이므로 별도 PR로 분리되었어야 했음.
이번 PR에서는 시간 제약으로 같이 머지되었으나, 후속 정리 필요.
현재 위치 (정리 대상)
- `POST /api/v1/feeds/{feedId}/early-start` 컨트롤러/서비스
- `POST /api/v1/feeds/{feedId}/early-start/consent` 컨트롤러/서비스
- `FeedItem.earlyStartRequested` 컬럼 + `requestEarlyStart()` / `canRequestEarlyStart()` 메서드
- `FeedApplication.earlyStartConsented` 컬럼 + `consentEarlyStart()` 메서드
- `FeedItemService.requestEarlyStart()` / `consentEarlyStart()`
작업
- `backend.feed.earlystart` 또는 `backend.feed.service.EarlyStartService` 로 책임 분리
- `FeedItemService` 에서 early-start 메서드 추출 → 새 서비스로 위임
- early-start 전용 단위 테스트 추가 (
EarlyStartServiceTest)
- Controller endpoint 이동/유지 결정 (REST 경로는 유지 권장)
우선순위
🟡 Med — 코드는 동작하지만 도메인 경계 정리 필요. PR #121 머지 후 진행.
참고
배경
PR #121 리뷰(hoTan35)에서 지적된 항목 — early-start 관련 코드는 acceptApplication 버그픽스(이슈 #120)와 별개 기능이므로 별도 PR로 분리되었어야 했음.
이번 PR에서는 시간 제약으로 같이 머지되었으나, 후속 정리 필요.
현재 위치 (정리 대상)
작업
EarlyStartServiceTest)우선순위
🟡 Med — 코드는 동작하지만 도메인 경계 정리 필요. PR #121 머지 후 진행.
참고