Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Jan 19, 2025

📄 요약(Summary)

Domain 레이어 exception 분리

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • Application 레이어에 대한 exception만 존재하여 이를 분리하고자 Domain 레이어 exception을 분리하였습니다.

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

  • error code도 레이어마다 분리를 할까 생각하였는데, 그렇게 하면 모든 유즈케이스마다 에러 코드를 생성해야할 수도 있을 것 같아 생각보다 복잡해질것 같아 기존대로 에러 코드를 유지하려고 합니다.

첨부 자료

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #26

@joowojr joowojr added ✨ feature 구현·개선 사항에 관련된 내용입니다 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다. labels Jan 19, 2025
@joowojr joowojr self-assigned this Jan 19, 2025
@joowojr joowojr merged commit 890b5d7 into develop Jan 19, 2025
@joowojr joowojr deleted the CLAP-67 branch January 19, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-67 도메인 레이어에 대한 예외 처리 구현

2 participants