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] review가 없는경우 delete review, delete worry transaction불가 수정 #82

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

leGit-y
Copy link
Contributor

@leGit-y leGit-y commented Nov 22, 2023

🔎 관련이슈

✨ 변경사항

수정전: worryRepository.ts 에서의 deleteWorry 함수호출
문제: 해당 함수는 리뷰삭제와 고민글 삭제를 하나의 transaction으로 묶고 있었음.
=> 때문에 리뷰가 없는 고민글의 경우, 리뷰삭제부분에서 오류 발생
수정후: deleteWorryWithReview, deleteWorryWithoutReview 함수로 분리

📃 참고사항

@leGit-y leGit-y added the Fix Something needs to be fixed label Nov 22, 2023
@leGit-y leGit-y merged commit df4adc8 into main Nov 22, 2023
1 check passed
@leGit-y leGit-y deleted the fix/deleteWorry branch November 22, 2023 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Something needs to be fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant