Merged
Conversation
minsoo0506
approved these changes
Mar 10, 2026
Collaborator
minsoo0506
left a comment
There was a problem hiding this comment.
안그래두 아까 읽어보구 있었는데...ㅎ 고생많았어 유찬 :)
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
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.
파일 구조 변경
DDD 기반으로 각 Domain 안에 controller , domain, dto, repository, service 가 들어가도록 변경했습니다.
common의 경우, 공통되거나 아예 관련성이 없을 경우 (TestController) 를 모아뒀습니다.
infrastructure, application 같이 DDD 구조를 세분화해서 짤 수도 있었지만, (실제로 그렇게도 잡아봄) 제 기준에서도 이해가 잘 안 되는데 굳이 이렇게 할 필요가 있을까 라는 마음에 간단하게 controller, service, repository 같은 형식을 유지했습니다.
개발자유미 센세의 https://www.youtube.com/watch?v=VOrA9Dfn3vY&t=562s 영상을 보고 제 입맛대로 잡아놓긴 했습니다.
만약 해당 구조가 별로거나, 더 나은 구조가 있다면 편하게 이야기해주세용
+3월 10일자 패치