[Fix] 멘토 로드맵 수정/삭제 오류 해결 #123
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
luckhee
pushed a commit
that referenced
this pull request
Oct 1, 2025
* fix: 멘토 로드맵 수정/삭제 오류 해결 및 로직 최적화 * chore: 로드맵 초기화 로직 비활성화 * feat: stepOrder 검증 로직 수정 및 테스트코드 수정
luckhee
added a commit
that referenced
this pull request
Oct 1, 2025
* LazyInitalizationException Transactional 추가 * PostCommentController Autowired삭제 후 @requiredargsconstructor로 교체 * PostComment 채택 시 Post의 isResolve state 병경 * refactor : Dto 에서 memberId 제거 * refactor : Dto 에서 memberId 제거 * refactor : 게시글 좋아요 -> 싫어요 토글 기능 Transactional 추가 * Test: 멘토링 연관 도메인 단위 테스트 도입 (#116) * Feat: 멘토링 관련 fixture 생성 * Test: 멘토링 단위 테스트 추가 * Test: 멘토 슬롯 단위 테스트 추가 * Test: 예약 단위 테스트 추가 * [Feat] 멘토 로드맵 샘플 데이터 추가 (#117) * feat: 멘토 로드맵 샘플 데이터 추가 * feat: 멘토 로드맵 샘플 데이터 추가 * feat: 멘토 로드맵 샘플 데이터 보강 * chore: 원활한 테스트를 위해 일부 로직 주석 처리 * Refactor: 멘토링 도메인에 멘토 닉네임 필드 반영 (#118) * Refactor: name -> nickname 반영 * Refactor: 검증 수정 * Refactor: 반복 시작, 종료 검증 추가 * fix: 멘토 멘티 맴버 fetch join하여 조회 * Fix: StringListConverter에서 JSON 이중 인코딩 문제 해결 * [Fix] 멘토 로드맵 수정/삭제 오류 해결 (#123) * fix: 멘토 로드맵 수정/삭제 오류 해결 및 로직 최적화 * chore: 로드맵 초기화 로직 비활성화 * feat: stepOrder 검증 로직 수정 및 테스트코드 수정 * [Feat] 예약 확정/취소/변경 (#125) * Feat: 예약 수락 * Fix: 하드코딩된 일정 -> now 기준으로 변경 * Feat: 예약 엔티티 검증 * Refactor: 예약 수락 시 동시성 제어 * Feat: 예약 거절, 취소 * Feat: 직업 로드맵 조회 api 구현 (#127) * feat: 멘토 로드맵 수정 * refactor: 권한 검증 로직 변경 * chore: 주석 수정 * feat: 직업 로드맵 조회 api 구현 * fix: 실패하는 테스트 수정 * refactor: 직업 로드맵 엔티티에서 @Setter 제거, 생성자에 빌더 패턴 적용 * feat: JobRoadmapNodeStat 필드 추가 및 직업 ID로 멘토 로드맵 조회 메서드 추가 * refactor: Response dto 수정 * feat: 직업 로드맵 조회 api 테스트용 샘플 데이터 추가 * chore: initSampleJobRoadmap() 비활성화 * member name -> member nickname으로 변경 * CommentModifyRequest record로 변경 * CommentDeleteRequest record로 변경 * CommentAllResponse, CommentCreateRequest record로 변경 * Dto record로 교체
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📢 기능 설명
멘토 로드맵 수정/삭제 시 clearNodes()가 호출되며 https://github.com/joincolumn 관계를 해제하려고 시도하면서 roadmap_id를 NULL로 업데이트하면서 DB 제약조건 위반 오류 발생
-> JPA Repository를 통한 직접 삭제로 로직 수정
stepOrder 검증 로직 수정
로직 변경에 맞춰 테스트코드 수정
필요시 실행결과 스크린샷 첨부
연결된 issue
연결된 issue를 자동으로 닫기 위해 아래 {이슈넘버}를 입력해주세요.
close #122
🩷 Approve 하기 전 확인해주세요!
✅ 체크리스트