Skip to content

Feat/72 issue label#76

Merged
meraki6512 merged 2 commits into
IssueDiver:devfrom
meraki6512:feat/72-issue-label
Sep 5, 2025
Merged

Feat/72 issue label#76
meraki6512 merged 2 commits into
IssueDiver:devfrom
meraki6512:feat/72-issue-label

Conversation

@meraki6512
Copy link
Copy Markdown
Contributor

연관된 이슈

#72


작업 내용

이슈 라벨 매핑 관련 (조회, 수정 기능) 오류를 수정했습니다.

  • IssueService의 라벨 업데이트 로직 수정: JPA의 암시적인 Cascade 방식 대신 IssueLabelRepository를 직접 사용해 명시적인 삭제 후 새로운 연결 저장
  • IssueLabelRepository 수정: deleteByIssueId를 추가하고 @Modifying(clearAutomatically = true) 어노테이션을 추가해 초기화 설정
  • IssueLabel/IssueLabelId 엔티티 수정: 생성자 및 필드 초기화
  • DB 스키마 마이그레이션 추가 (V3): 누락되었었던 issue_label 테이블의 added_at 컬럼을 추가하는 V3 마이그레이션 스크립트 추가
  • IssueServiceTest 수정: 리팩토링 후 관련 테스트 코드 수정 반영

📸 스크린샷 (선택)

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?


PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).
  • main branch가 아닌 dev branch에 PR 요청을 했습니다. (main branch에 바로 PR&merge하지 않기).

- IssueService의 라벨 업데이트 로직 수정: JPA의 암시적인 Cascade 방식 대신 IssueLabelRepository를 직접 사용해 명시적인 삭제 후 새로운 연결 저장
- IssueLabelRepository 수정: deleteByIssueId를 추가하고 @Modifying(clearAutomatically = true) 어노테이션을 추가해 초기화 설정
- IssueLabel/IssueLabelId 엔티티 수정: 생성자 및 필드 초기화
- DB 스키마 마이그레이션 추가 (V3): 누락되었었던 issue_label 테이블의 added_at 컬럼을 추가하는 V3 마이그레이션 스크립트 추가
- 이슈 라벨 수정 기능(updateIssue) 및 단건 조회(getIssue) 로직 변경으로 인해 실패하던 IssueServiceTest의 단위 테스트들 수정
@meraki6512 meraki6512 merged commit 6d2c5ce into IssueDiver:dev Sep 5, 2025
@meraki6512 meraki6512 deleted the feat/72-issue-label branch September 5, 2025 05:18
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