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

[FEAT] ProfileController Facade & DCI 패턴 #243

Merged
merged 1 commit into from
Aug 14, 2024
Merged

Conversation

kimdozzi
Copy link
Contributor

PR 타입(하나 이상의 PR 타입을 선택해주세요)

☑ 기능 추가

□ 기능 삭제

□ 버그 수정

□ 의존성, 환경 변수, 빌드 관련 코드 업데이트


반영 브랜치

feat/241profile-facade -> main


변경 사항

  • ProfileService 제거
  • ProfileFacade & ProfileFacadeService 구현
  • 테스트 코드 DCI 패턴 도입

테스트 결과

image


연관된 이슈

#241


리뷰 요구사항(선택)

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

- ProfileService 제거
- ProfileFacade & ProfileFacadeService 구현
- 테스트 코드 DCI 패턴 도입
@kimdozzi kimdozzi added 💡feature 기능 구현, 개선 사항 관련 ⚙️ refactor 리팩토링 🧑🏻‍💻 BE 백엔드 관련 코드 🖥️ server 서버 관련 🎮 test 테스트 코드 관련 labels Aug 12, 2024
@kimdozzi kimdozzi self-assigned this Aug 12, 2024
@kimdozzi kimdozzi linked an issue Aug 12, 2024 that may be closed by this pull request
3 tasks
Copy link

Test Results

166 files  166 suites   32s ⏱️
295 tests 295 ✅ 0 💤 0 ❌
303 runs  303 ✅ 0 💤 0 ❌

Results for commit adb61b6.

Copy link
Contributor

@SSung023 SSung023 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 !! :)

@kimdozzi kimdozzi merged commit 8eb7648 into main Aug 14, 2024
2 checks passed
@kimdozzi kimdozzi deleted the feat/241profile-facade branch August 25, 2024 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑🏻‍💻 BE 백엔드 관련 코드 💡feature 기능 구현, 개선 사항 관련 ⚙️ refactor 리팩토링 🖥️ server 서버 관련 🎮 test 테스트 코드 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Profile Facade & DCI
2 participants