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-21] hotfix : 미션 재인증(삭제) 시 500에러 해결 #284

Merged
merged 1 commit into from
May 15, 2024

Conversation

seungueonn
Copy link
Member

PR 타입

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

개요

미션 재인증(삭제) 시 연관관계에 있는 missionComment 삭제 로직 추가

변경 사항

  • MissionArchive DB 삭제하려고 하던중 연관관계에 있는 MissionComments 가 있어 삭제할 수 없다는 에러를 발견하였습니다.
  • missionArchive 가 같은 missionComment 들을 모두 삭제하는 datajpa 메소드를 추가하고 이를 사용한 로직을 추가하였습니다.

코드 리뷰 시 참고 사항

public void deleteAllCommentByMissionArchive(Long missionArchiveId) {
missionCommentRepository.deleteAllMissionCommentsByMissionArchiveId(missionArchiveId);
}

void deleteAllMissionCommentsByMissionArchiveId(Long missionArchiveId);

테스트 결과

@seungueonn seungueonn changed the title [MNG-21] hotfix : 미션 재인증(삭제) 시 연관관계에 있는 missionComment 삭제 로직 추가 [MNG-21] hotfix : 미션 재인증(삭제) 시 500에러 해결 May 15, 2024
@minsu20
Copy link
Member

minsu20 commented May 15, 2024

mission Archive삭제시 미션 댓글도 삭제하는 로직을 추가한건가요?

@seungueonn
Copy link
Member Author

네 맞습니다! 그래서 missonArchiceDeleteService의 deleteMissionArhice 메소드에 로직을 추가할까도 생각했었는데 그러면 domain service 기준이 깨지는 것 같아 지금처럼 usecase 로직에 추가했어요! 어떤게 좋을까요?

@minsu20
Copy link
Member

minsu20 commented May 15, 2024

유스케이스에 있는게 좋은 것 같아요

@minsu20 minsu20 merged commit 538e78a into main May 15, 2024
1 check passed
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.

2 participants