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

fix : 알림 같이 가기 요청이 이미 존재한 경우 다시 알림을 보낼 수 없도록 validation 추가 #255

Conversation

java-saeng
Copy link
Collaborator

#️⃣연관된 이슈

📝작업 내용

알림 가기 요청이 이미 있을 때, 다시 알림 요청을 하면 exception이 터지는 validation 추가

예상 소요 시간 및 실제 소요 시간

3시간 / 3시간

@java-saeng java-saeng added 버그 개발자가 의도하지 않은 상황 Backend 백엔드 관련 이슈 High Priority 리뷰 우선순위가 높은 PR labels Aug 8, 2023
@java-saeng java-saeng added this to the 4차 스프린트 1주차 milestone Aug 8, 2023
@java-saeng java-saeng self-assigned this Aug 8, 2023
Copy link
Collaborator

@hyeonjerry hyeonjerry left a comment

Choose a reason for hiding this comment

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

크게 수정할 점은 보이지 않습니다.
수고하셨습니다.

Copy link
Collaborator

@amaran-th amaran-th left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!
크게 수정할 부분은 보이지 않네요~!

@java-saeng java-saeng merged commit 0f60c4a into backend-main Aug 8, 2023
1 check passed
@hyeonjerry hyeonjerry deleted the Fix/#220-같이가요_요청을_다시_보낼_수_없게_수정 branch August 17, 2023 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend 백엔드 관련 이슈 High Priority 리뷰 우선순위가 높은 PR 버그 개발자가 의도하지 않은 상황
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants