Skip to content

Conversation

@7zrv
Copy link
Collaborator

@7zrv 7zrv commented Dec 8, 2024

📌 과제 설명

기관의 자신이 등록한 선호물품 삭제 기능 구현입니다.

👩‍💻 요구 사항과 구현 내용

  • 선호 물품 레포지토리의 인터페이스화 리팩토링
  • 리팩토링에 의한 의존성 경로 수정
  • 삭제 쿼리 메서드 구현
  • 삭제 서비스 메서드 구현
  • 예외 처리 메서드 구현
  • 예외 메세지 추가
  • 컨트롤러 구현

✅ PR 포인트 & 궁금한 점

7zrv added 4 commits December 8, 2024 16:22
- 기존 Jpa 단일 레포지토리에서 인터페이스, 구현체 Jpa 레포지토리로 분할
- 분할에 따른 의존성 경로 수
- 권한, 파일 없음에 대한 예외 메세지 추가
- 서비스 메서드 구현
- 예외처리 추가
- 예외 처리를 위한 쿼리 메서드 추가
- 컨트롤러 구현
- 테스트 코드 작성및 검증 완료
@7zrv 7zrv self-assigned this Dec 8, 2024
@7zrv 7zrv linked an issue Dec 8, 2024 that may be closed by this pull request
3 tasks
github-actions[bot]

This comment was marked as off-topic.

@github-actions github-actions bot changed the title Feature/197 기관의 선호물품 삭제 기능 구현 [BUILD FAIL] Feature/197 기관의 선호물품 삭제 기능 구현 Dec 8, 2024
@github-actions github-actions bot closed this Dec 8, 2024
- 중복 아이디 사용으로 인한 테스트 실패를 수정하기 위해 @beforeeach를 이용한 테스트 실행전 데이터 클렌징 작업 수행 추
@7zrv 7zrv reopened this Dec 8, 2024
@7zrv 7zrv changed the title [BUILD FAIL] Feature/197 기관의 선호물품 삭제 기능 구현 Feature/197 기관의 선호물품 삭제 기능 구현 Dec 8, 2024
Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다. 패키지 변경 내역들도 확인했습니다!

return center;
}

} No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

개행이 빠졌습니다~

Copy link
Collaborator

@leebs0521 leebs0521 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 !

- 잠재적 에러 예방을 위한 개행 추가
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 8, 2024

@7zrv 7zrv merged commit b23e334 into main Dec 8, 2024
3 checks passed
@7zrv 7zrv deleted the feature/197-add-preferitem-delete branch December 8, 2024 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]기관의 선호물품 삭제 기능 구현

4 participants