Skip to content

[Refactor] status 누락 파트 수정 - #80

Merged
githyj-jang merged 6 commits into
devfrom
refactor/34-status
May 20, 2026
Merged

githyj-jang merged 6 commits into
devfrom
refactor/34-status

Conversation

@githyj-jang

Copy link
Copy Markdown
Contributor

📝 작업 내용

이번 PR에서 작업한 내용을 설명해주세요.

  • 상태 변경 책임 분리 온전하지 못한 부분 수정

🚀 주요 변경 사항

완료한 이슈 번호
Close #
관련된 이슈 번호 (닫고 싶지 않은 경우)
Related to #

✅ 자체 체크리스트 (필수)

  • ./gradlew build 실행 결과 정상 (인증샷 첨부)
  • IntelliJ HTTP Client 테스트 완료 (인증샷 첨부)
  • 팀 내 컨벤션 준수 및 불필요한 로그, import 제거
  • 중요한 변경 사항이 팀에 공유되었는지

📸 테스트 인증샷

빌드 결과 및 IntelliJ HTTP Client 실행 화면을 여기에 첨부해 주세요.

💬 리뷰어 전달사항 (선택)

특별히 봐주었으면 하는 부분이나 논의가 필요한 점을 적어주세요.

  • 논의점


📎 참고 자료

관련 문서, 레퍼런스 링크 등이 있다면 여기에 첨부해주세요.

@githyj-jang githyj-jang self-assigned this May 20, 2026
Copilot AI review requested due to automatic review settings May 20, 2026 15:46
@coderabbitai

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Reservation 상태 전이/정책 판단 로직을 Reservation 엔티티에서 상태 객체(ReservationState)로 더 이동시켜 책임 분리를 보완하고, 운영 환경 시간대 및 배포 파이프라인 설정을 함께 조정하는 PR입니다.

Changes:

  • Reservation의 완료/수정 가능 여부 및 체크인 가능 시간 검증을 ReservationState/ConfirmedState로 위임
  • 기존 ReservationTransition(enum) 및 관련 테스트 제거
  • Asia/Seoul 시간대 설정 추가 및 CI/CD 워크플로우 구조/환경변수 주입 방식 변경

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/main/java/com/michelet/reservation/domain/state/ReservationState.java 상태별 정책/검증 메서드 기본 구현 추가(비-CONFIRMED는 false/예외)
src/main/java/com/michelet/reservation/domain/state/ConfirmedState.java 취소/수정 가능, 수정/완료 가능 assert 로직을 CONFIRMED 상태로 이동
src/main/java/com/michelet/reservation/domain/entity/Reservation.java complete/modify/isCancellable/isModifiable 로직을 상태 객체로 위임
src/main/resources/application.yaml Jackson 기본 time-zone을 Asia/Seoul로 설정
Dockerfile JVM 기본 timezone을 Asia/Seoul로 강제
.github/workflows/ci-prod.yml main 브랜치 기준으로 테스트/빌드푸시 잡 분리 및 권한/조건 정리
.github/workflows/cd-prod.yml ECS task definition에 TZ 및 DB/Redis/internal auth 관련 env 주입 추가
src/main/java/com/michelet/reservation/domain/enums/ReservationTransition.java 전이 enum 제거
src/test/java/com/michelet/reservation/domain/enums/ReservationTransitionTest.java 전이 enum 테스트 제거

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci-prod.yml
Comment thread .github/workflows/ci-prod.yml
Comment thread .github/workflows/cd-prod.yml
@githyj-jang
githyj-jang merged commit 20db926 into dev May 20, 2026
3 checks 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.

2 participants