Skip to content

Conversation

jaewoo9797
Copy link
Collaborator

@jaewoo9797 jaewoo9797 commented Aug 8, 2025

📌 PR 제목

간결하고 명확하게 작성해주세요

api 모듈에 존재하는 도메인 엔티티를 분리해서 closet-domain 모듈을 생성하고 이전했습니다.

비즈니스 로직에 필요한 로직 추가

🔗 관련 이슈 번호

#18

📝 변경 사항 요약

무엇을, 왜 변경했는지 간략히 작성해주세요

도메인 모듈은 다른 모듈을 의존하지 않으면서 생기는 문제로, api 모듈에 존재하는 dto나 예외 클래스를 임포트할 수 없음. 코드 로직을 수정 또는 파라미터 수정을 해주었습니다.

기존 엔티티 중에 관리되어야할 생성시간, 수정 시간이 필요한 엔티티에 컬럼을 추가해주고 로컬과 운영 디비에 적용해주었습니다.

  • 적용 시 두 번 해야 하고(운영, 개발 디비) 여러 테이블의 컬럼이 수정되어야 하면 휴먼 에러 발생 가능성이 높아질 것으로 예상함
  • 해결책으로 Flyway 같은 데이터베이스 마이그레이션 기능을 이용 필요 느낌 -> 이후 적용 방법 조사 후 적용할 예정

중계 테이블에 필요없는 컬럼이 존재함. 주석으로 리팩토링 TODO 추가
OneToMany 컬럼에서 데이터 cascade 설정 수정이 필요해 보이는 곳에 TODO 추가

✅ 체크리스트

  • 코드 빌드 및 컴파일 통과
  • 단위/통합 테스트 통과
  • 문서/주석 업데이트 완료

@jaewoo9797 jaewoo9797 self-assigned this Aug 8, 2025
@jaewoo9797 jaewoo9797 added the enhancement New feature or request label Aug 8, 2025
@jaewoo9797 jaewoo9797 merged commit 1f7c73a into dev Aug 12, 2025
1 check passed
@jaewoo9797 jaewoo9797 deleted the feat/#18 branch August 12, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant