Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat : 예약 조회 로직 구현 #45

Merged
merged 11 commits into from
Jan 3, 2024
Merged

Conversation

dlswns2480
Copy link
Member

close #44

⛏ 작업 상세 내용

  • 예약 조회 쿼리 구현
    • 예약 조회 시 예약시간과 매장을 한번의 쿼리로 가져오도록 페치 조인 활용
  • 서비스 계층 예약 조회 로직 구현 (ReservationService에 대해서만 구현)
  • 예약 생성자 수정
    • 기존 : 매장, 예약시간을 인자로 받아 각각 필드에 주입
    • 변경 : 예약시간만 인자로 받아 매장 필드에는 예약시간의 매장을 객체 탐색을 통해 주입
    • 예약시간이 매장을 참조하고 있는데 따로 받을 필요가 없어서 수정했습니다.
  • api 스펙 수정
    • 응답 body에 예약 상태 필드가 없어 그것만 추가했고 노션에 반영했습니다!
  • repository, service 테스트

📝 작업 요약

  • 예약 조회 로직 구현, 단위 테스트

☑️ 중점적으로 리뷰 할 부분

  • 쿼리, 비즈니스 로직, 테스트코드, 메소드 네이밍

@dlswns2480 dlswns2480 self-assigned this Jan 2, 2024
Copy link

github-actions bot commented Jan 2, 2024

Test Results

22 tests  +3   22 ✅ +3   5s ⏱️ ±0s
11 suites +1    0 💤 ±0 
11 files   +1    0 ❌ ±0 

Results for commit 5c9b5e6. ± Comparison against base commit b35b28f.

♻️ This comment has been updated with latest results.

@dlswns2480 dlswns2480 merged commit 4b3b295 into dev Jan 3, 2024
3 checks passed
@dlswns2480 dlswns2480 linked an issue Jan 5, 2024 that may be closed by this pull request
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

feature : 예약 전체 조회 로직 구현
3 participants