feat : 직관일지 없이 좌석시야 독립 등록 지원 #142

[GOOHAESEUNG]

Closes #142

변경 내용

• journalId 선택값으로 변경 (없으면 일지와 무관하게 독립 등록)
• journalId 있는 경우 기존과 동일하게 일지 연결 + 중복 등록 체크
• 응답 JournalId는 일지 없을 경우 null 반환

수정 파일

• SeatViewService.java — journalId null 분기 처리
• SeatCreateResDto.java — journal null 시 NPE 방지
• SeatCreateReqDto.java — journalId 선택값 Swagger 명시
• SeatViewController.java — API 설명 업데이트

Summary by CodeRabbit

주요 변경사항

• 기능 개선
  • 좌석 시야 생성 시 저널 연결이 이제 선택사항입니다. 저널과의 연결 없이도 좌석 시야를 독립적으로 등록할 수 있으므로, 더 유연하고 편리한 좌석 시야 관리가 가능합니다.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9021168e-6d16-464e-8477-b6abc7ffa7e2

📥 Commits

Reviewing files that changed from the base of the PR and between c87cf38 and b7c2592.

📒 Files selected for processing (4)

• src/main/java/com/inninglog/inninglog/domain/seatView/controller/SeatViewController.java
• src/main/java/com/inninglog/inninglog/domain/seatView/dto/req/SeatCreateReqDto.java
• src/main/java/com/inninglog/inninglog/domain/seatView/dto/res/SeatCreateResDto.java
• src/main/java/com/inninglog/inninglog/domain/seatView/service/SeatViewService.java

---

📝 Walkthrough

Walkthrough

직관일지와의 연관을 선택사항으로 변경하는 기능 개선입니다. journalId를 필수 입력값에서 선택 입력값으로 재분류하고, 일지가 없어도 좌석시야를 독립적으로 등록할 수 있도록 수정했습니다. API 문서, DTO 스키마, 서비스 로직, 응답 구조가 이 변경사항을 반영하도록 일관되게 업데이트되었습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

검토 포인트:

• DTO 스키마: journalId nullable 설정이 일관되게 적용되었는지 확인 필요
• 서비스 로직: createSeatView()의 조건부 처리가 모든 케이스를 정확히 다루는지 검증 필요
  • journalId가 있을 때: 기존 로직과 동일하게 동작하는지
  • journalId가 없을 때: 일지 조회/검증을 건너뛰고 정상 작동하는지
  • 특히 journal.setSeatView(seatView) 호출이 journal이 non-null일 때만 실행되는지
• 응답 구조: SeatCreateResDto에서 journalId가 null로 반환되는 경우가 응답 사양과 일치하는지 확인

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/#142/seatview-without-journal

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}