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-63 [fix] 주점 대기인원 조회 API 및 주점관리 API 수정 #65

Merged
merged 8 commits into from
Jul 21, 2024

Conversation

jwnnoh
Copy link
Member

@jwnnoh jwnnoh commented Jul 19, 2024

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

  • 주점 마감 후 재오픈시 기존 대기열을 초기화하기 위함입니다.
    • 이를 주점에 할당된 유효한 대기열(입장 가능, 대기중)을 모두 취소처리함으로서 수정하였습니다.
  • 커스텀 예외처리가 아닌 예외를 JSON으로 반환할 수 있도록 핸들러를 추가하였습니다.
  • 주점 상세 조회 시 헤더에서 커스텀 인증 객체를 가져올 수 있도록 수정하였습니다.
  • 메뉴 수정 시 중복되는 객체가 생성되지 않도록 수정하였습니다.

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


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

Before

스크린샷 2024-07-19 오후 7 31 40

After

스크린샷 2024-07-19 오후 7 31 44 스크린샷 2024-07-19 오후 7 31 55

4. 완료 사항

  • 대기열 초기화
  • 메뉴 중복 생성 수정

5. 추가 사항

  • 예외처리 핸들러 추가

@jwnnoh jwnnoh requested a review from rootTiket July 19, 2024 14:33
@jwnnoh jwnnoh self-assigned this Jul 19, 2024
@jwnnoh jwnnoh requested review from yechan-kim and removed request for rootTiket July 19, 2024 14:33
Comment on lines 52 to 53
menuRepository.save(menu);
return menu;
Copy link
Contributor

Choose a reason for hiding this comment

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

JPA의 save()는 저장한 객체를 그대로 리턴하기 때문에 아래처럼 변경하는 것이 더 좋을 것 같습니다.

return menuRepository.save(menu);

Copy link
Member Author

Choose a reason for hiding this comment

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

해당 부분 생각을 못하고 있었습니다. 감사합니다!! 👍

Copy link
Contributor

@yechan-kim yechan-kim Jul 20, 2024

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

고생하셨습니다! 의견 확인 부탁드립니다!

@jwnnoh jwnnoh merged commit 6309a5d into develop Jul 21, 2024
@jwnnoh jwnnoh deleted the GTB-63 branch July 21, 2024 07:23
@jwnnoh jwnnoh added the 🐛 bug 버그 리포트 label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug 버그 리포트
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants