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

GTB-71 [refactor] 웨이팅 신청 서비스 정책 변경 #80

Merged
merged 7 commits into from
Sep 4, 2024
Merged

Conversation

jwnnoh
Copy link
Member

@jwnnoh jwnnoh commented Sep 4, 2024

1. 무슨 이유로 코드를 변경했나요?

웨이팅 신청 정책 변경

현재 사용자가 특정 주점을 이용중일 경우, 해당 시간동안 웨이팅 신청을 막아야 합니다.

  • Waiting 도메인에 User 필드를 추가하였습니다.
  • 추가한 User필드로 사용자의 Seating 도메인 리스트를 확인합니다.
  • 만약 exitTime이 현재 시간보다 뒤에 있다면 예외를 발생시키도록 하였습니다.


2. 어떤 위험이나 장애를 발견했나요?

  • �강제 취소 알림톡이 현재 2번 발생되는 오류를 파악하여 수정 예정입니다.

3. 관련 스크린샷을 첨부해주세요.

스크린샷 2024-09-04 오후 1 43 02

4. 완료 사항

  • 웨이팅 정책 변경

이슈 번호

close #79


5. 추가 사항

  • TODO를 수정 및 추가하였습니다.
  • User 도메인에 BaseTimeEntity 추가

@jwnnoh jwnnoh added the ♻️ refactor 코드 리팩토링 label Sep 4, 2024
@jwnnoh jwnnoh self-assigned this Sep 4, 2024
@jwnnoh jwnnoh requested review from yechan-kim, hoonyworld and rootTiket and removed request for yechan-kim and hoonyworld September 4, 2024 04:51
Copy link
Contributor

@rootTiket rootTiket left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 크게 문제는 없어보입니다 ! 이미 이용 중인 경우 예외처리도 꼼꼼하게 되어 있어 요구사항도 충분히 충족되네요 :)

@yechan-kim
Copy link
Contributor

LGTM 👍

@jwnnoh jwnnoh changed the title Gtb 71 GTB-71 [refactor] 웨이팅 신청 서비스 정책 변경 Sep 4, 2024
@jwnnoh jwnnoh merged commit c8c4509 into develop Sep 4, 2024
@jwnnoh jwnnoh deleted the GTB-71 branch September 4, 2024 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GTB-71 [refactor] 웨이팅 신청 서비스 정책 변경
3 participants