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-47 [feat] 웨이팅 취소 API 구현 #48

Merged
merged 6 commits into from
Jul 9, 2024
Merged

GTB-47 [feat] 웨이팅 취소 API 구현 #48

merged 6 commits into from
Jul 9, 2024

Conversation

jwnnoh
Copy link
Member

@jwnnoh jwnnoh commented Jul 8, 2024

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

  • 사용자가 웨이팅을 취소하기 위한 로직을 구현하기 위함입니다.

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

  • POSTMAPPING으로 요청이 진행되어, updatedAt이 갱신되지 않는 오류를 발견하여 PATCHMAPPING으로 수정하였습니다.
  • updatedAt이 수정되지 않는 오류를 파악, BaseTimeEntity 클래스에 updatable이 false로 되어있는 부분을 발견하고 수정하였습니다.

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

스크린샷 2024-07-09 오전 2 11 19 스크린샷 2024-07-09 오전 2 12 23

4. 완료 사항

  • 웨이팅 취소 API 구현

5. 추가 사항

  • 웨이팅 취소 API는 사용자가 웨이팅 목록에서 확인하고 취소하는 로직입니다.
  • 이에 웨이팅 목록을 반환할 때, waitingId도 동시에 반환하도록 하였습니다.

@jwnnoh jwnnoh added the ✨ feature 새로운 기능 label Jul 8, 2024
@jwnnoh jwnnoh self-assigned this Jul 8, 2024
@jwnnoh jwnnoh requested review from yechan-kim and rootTiket and removed request for yechan-kim July 8, 2024 17:16
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.

LGTM 👍

@jwnnoh jwnnoh merged commit 5d6903d into develop Jul 9, 2024
@jwnnoh jwnnoh deleted the GTB-47 branch July 9, 2024 03:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 새로운 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants