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] 예약 현황 조회 기능 구현 및 단위 테스트 작성 #10

Merged
merged 12 commits into from
Dec 3, 2024

Conversation

uijin31
Copy link
Collaborator

@uijin31 uijin31 commented Dec 3, 2024

🔗 관련 이슈

Resolves #9

👩🏻‍💻 구현 내용

  • H2, Spring Data Jpa 의존성 추가
  • Booking, BookingSlot 엔티티 생성
  • 예약 현황 조회 기능 구현
  • 예약 현황 조회 단위 테스트 작성
  • 예약 현황 조회 통합 테스트 수정

예약 ERD

스크린샷 2024-12-03 18 16 02

예약 슬롯 ERD

스크린샷 2024-12-03 18 15 12

💬 고민되는 점

  • 예약 슬롯에 isBooked 컬럼이 사실 데이터 중복인데, 편의를 위해 반정규화를 했습니다! 정규화를 어디까지 진행해야 하는지 고민이 되네요..
  • 예약이 승인 대기 중 / 결제 대기 중일때도 일단 예약된걸로 하는 것이 좋겠죠?
  • 엔티티(모델) 간에 의존성이 줄이는 것이 추후 확장에서 도움이 될 것 같아, FK를 두지 않았습니다! 혹시 이 방식에 대해 의견을 들을 수 있을까요?

@uijin31 uijin31 added the ✨ feature New feature or request label Dec 3, 2024
@uijin31 uijin31 self-assigned this Dec 3, 2024
@uijin31 uijin31 merged commit a96e0cf into feat/#8-develop-booking Dec 3, 2024
1 check passed
@f-lab-lyan
Copy link
Collaborator

음?? 이 PR이 머지가 된 이유는...???

@uijin31
Copy link
Collaborator Author

uijin31 commented Dec 3, 2024

feat/#8-develop-booking 이 브랜치에 머지하고.. 기능 2개정도 모이면 main으로 PR 날리면서 리뷰 받으려고 했습니다..!!😂
다시 열어 놓는게 좋을까요?

@f-lab-lyan
Copy link
Collaborator

아니오 괜찮습니다. :-) 다만, 다음부턴 코멘트 하나만 남겨주세요. ㅎㅎ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants