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

refactor: 플랜 도메인 퍼샤드를 통한 유스케이스 통합 리팩토링 #105

Merged
merged 6 commits into from
Jan 16, 2025

Conversation

junseoparkk
Copy link
Member

📝 개요

  • 플랜 도메인 퍼샤드를 통한 유스케이스 통합 리팩토링
  • 스웨거 api를 위한 PlanApi 추가

✨ 변경 사항

  • ✨ 기존 플랜 퍼샤드 서비스를 조회 및 커맨드 퍼샤드로 분리
  • ✨ 퍼샤드 분리로 인한 요청/응답 클래스 구조 및 형식 변경
  • ✨ 스웨거 api를 위한 PlanApi 인터페이스에 메서드 추가

🔗 관련 이슈

ℹ️ 참고 사항

  • 최대한 타 도메인과의 컨벤션과 통일하기 위해 노력했습니다.
  • 기존 하나의 퍼샤드 서비스를 조회와 커맨드 요청에 따라 두 개의 퍼샤드로 분리하였습니다.
  • 스웨거 api 설정을 위한 인터페이스 내 메서드를 추가했습니다.
  • 포스트맨을 통한 기본 CRUD 기능은 테스트 완료하였습니다.

@junseoparkk junseoparkk added refactor 로직 리팩토링 feature 기능 개발 labels Jan 16, 2025
@junseoparkk junseoparkk self-assigned this Jan 16, 2025
Copy link

Test Results

7 tests   7 ✅  0s ⏱️
9 suites  0 💤
9 files    0 ❌

Results for commit e262e88.

Copy link
Member

@JiHongKim98 JiHongKim98 left a comment

Choose a reason for hiding this comment

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

임시 레이어 컨벤션이긴 하지만 컨벤션 맞추는 방향으로 추가한거 너무 좋아유~

Copy link
Member

@Namgyu11 Namgyu11 left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다~~
LGTM😎👍

@junseoparkk junseoparkk merged commit 4e4c14e into dev Jan 16, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 기능 개발 refactor 로직 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor: 플랜 도메인 퍼샤드 서비스를 통한 유스케이스 통합 리팩토링
3 participants