Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GETP-203 refactor: client 컴포넌트 포트-어댑터 아키텍쳐 적용 #137

Merged
merged 8 commits into from
Aug 21, 2024

Conversation

scv1702
Copy link
Member

@scv1702 scv1702 commented Aug 21, 2024

✨ 구현한 기능

  • client 컴포넌트 포트-어댑터 아키텍쳐 적용
  • Mapper에서 source가 null임에도 내부 필드가 null인 target으로 매핑하는 오류 수정

📢 논의하고 싶은 내용

  • MapStruct 사용 시 꼭 구현체를 확인해야할 것 같아요. MapStruct가 좋은 도구이긴 하지만 매핑이 잘못된 순간 큰 장애로 이어질 수 있으니 조심합시다.

🎸 기타

Copy link

Test Results

139 tests  +6   139 ✅ +6   13s ⏱️ ±0s
 90 suites +2     0 💤 ±0 
 90 files   +2     0 ❌ ±0 

Results for commit 6bdfc8f. ± Comparison against base commit 6c60ddd.

@scv1702 scv1702 merged commit 9b0069f into develop Aug 21, 2024
2 checks passed
@scv1702 scv1702 deleted the refactor/GETP-203 branch August 21, 2024 19:15
scv1702 added a commit that referenced this pull request Sep 12, 2024
* GETP-203 refactor: client 컴포넌트의 application 모듈 port 생성

* GETP-203 refactor: ClientService를 use case에 따라 분리

* GETP-203 refactor: Address를 JPA VO와 도메인 VO로 분리

* GETP-203 refactor: BankAccount를 JPA VO와 도메인 VO로 분리

* GETP-203 refactor: Client를 JPA Entity와 도메인 Entity로 분리

* GETP-203 feat: RegisterClientService에서 LoadMemberPort에 대한 의존성 제거

* GETP-203 refactor: ProjectDao에서 Client 조회에 대한 책임을 ClientQuery로 이전

* GETP-203 fix: Mapper에서 source가 null임에도 내부 필드가 null인 target으로 매핑하는 오류 수정
scv1702 added a commit that referenced this pull request Sep 12, 2024
* GETP-203 refactor: client 컴포넌트의 application 모듈 port 생성

* GETP-203 refactor: ClientService를 use case에 따라 분리

* GETP-203 refactor: Address를 JPA VO와 도메인 VO로 분리

* GETP-203 refactor: BankAccount를 JPA VO와 도메인 VO로 분리

* GETP-203 refactor: Client를 JPA Entity와 도메인 Entity로 분리

* GETP-203 feat: RegisterClientService에서 LoadMemberPort에 대한 의존성 제거

* GETP-203 refactor: ProjectDao에서 Client 조회에 대한 책임을 ClientQuery로 이전

* GETP-203 fix: Mapper에서 source가 null임에도 내부 필드가 null인 target으로 매핑하는 오류 수정
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