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

MNG-01 탈퇴한 팀 인원 중복으로 반영되는 버그 수정 #211

Merged
merged 6 commits into from
Jan 29, 2024

Conversation

minsu20
Copy link
Member

@minsu20 minsu20 commented Jan 29, 2024

PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 업데이트
  • 기타 사소한 수정

개요

  • 팀을 탈퇴하는 API 를 호출할 때 인원이 반영이 안되는 버그를 수정

변경 사항

  • 기존에는 팀을 탈퇴할 때 해당 팀 멤버가 삭제된 것을 고려하지 않아서, 동일한 유저가 팀을 탈퇴하는 API를 2번 호출하면 팀 멤버 인원이 줄어드는 함수가 2번 호출이 되어서 팀 인원이 반영이 2번이 되는 오류가 있었음.
  • 따라서 팀을 탈퇴할 때 멤버를 조회하는 함수에서 삭제가 안되는 경우(isDeleted가 false)만 조회하게 수정, 이는 팀을 삭제할 때도 동일하게

코드 리뷰 시 참고 사항

  • mapper의 의존성을 지우기 위해 모두 static으로 바꾸었습니다.

@minsu20 minsu20 merged commit 2b7d4d9 into main Jan 29, 2024
1 check passed
@minsu20 minsu20 deleted the MNG-01 branch January 29, 2024 09:21
@minsu20 minsu20 mentioned this pull request Jan 30, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant