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] InstanceDetail Facade Pattern 적용 #242

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

kimdozzi
Copy link
Contributor

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

☑ 기능 추가

□ 기능 삭제

□ 버그 수정

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


반영 브랜치

feat/240-instance-detail-facade -> main


변경 사항

  • InstanceDetailFacade & InstanceDetailFacadeService 생성
  • InstanceDetailService 기능 상실 -> 제거
  • 테스트 코드 DCI 도입

테스트 결과

image


연관된 이슈

#240


리뷰 요구사항(선택)

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

- InstanceDetailFacade & InstanceDetailFacadeService 생성
- InstanceDetailService 기능 상실 -> 제거
- 테스트 코드 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

175 files  175 suites   33s ⏱️
295 tests 295 ✅ 0 💤 0 ❌
303 runs  303 ✅ 0 💤 0 ❌

Results for commit 732d6ef.

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 5ef6720 into main Aug 14, 2024
2 checks passed
@kimdozzi kimdozzi deleted the feat/240-instance-detail-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] InstanceDetail Facade & DCI
2 participants