Skip to content

feat(qna, progress): add DTOs, domain models, repositories, and use cases #38

Merged
Pochomo merged 12 commits intodevelopfrom
feature/#17
Mar 28, 2025
Merged

feat(qna, progress): add DTOs, domain models, repositories, and use cases #38
Pochomo merged 12 commits intodevelopfrom
feature/#17

Conversation

@sohyunxxi
Copy link
Collaborator

#️⃣ Issue Number

#17 스포티파이 질문 및 답변 이슈

📝 요약(Summary)

QnA 모듈의 기능을 구현하기 위해 다음과 같은 변경을 수행했습니다:

  • DTO 추가: 클라이언트와의 데이터 교환을 위해 QuestionDTOAnswerDTO 클래스를 생성했습니다.
  • 도메인 모델 추가: QuestionAnswer 엔티티 클래스를 정의하여 데이터베이스와의 매핑을 설정했습니다.
  • 레포지토리 추가: QuestionRepositoryAnswerRepository 인터페이스를 생성하여 데이터베이스 접근을 추상화했습니다.
  • 유스케이스 추가: QnAService 클래스를 작성하여 비즈니스 로직을 구현하고, 컨트롤러에서 호출할 수 있도록 했습니다.

이러한 변경을 통해 QnA 기능을 체계적으로 구현하고, 향후 유지보수와 확장을 용이하게 했습니다.

Progress 모듈의 진행 상황 추적 기능을 추가하기 위해 다음과 같은 변경을 수행했습니다:

  • DTO 추가: 진행 상황 데이터를 전송하기 위한 ProgressDTO 클래스를 생성했습니다.
  • 도메인 모델 추가: Progress 엔티티 클래스를 정의하여 데이터베이스와의 매핑을 설정했습니다.
  • 레포지토리 추가: ProgressRepository 인터페이스를 생성하여 진행 상황 데이터에 접근할 수 있도록 했습니다.
  • 유스케이스 추가: ProgressService 클래스를 작성하여 진행 상황 관련 비즈니스 로직을 구현했습니다.

이러한 변경을 통해 사용자의 진행 상황을 효과적으로 추적하고 관리할 수 있는 기능을 추가했습니다.

🛠️ PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📸스크린샷 (선택)

💬 공유사항 to 리뷰어

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

@sohyunxxi sohyunxxi added the enhancement New feature or request label Mar 26, 2025
@sohyunxxi sohyunxxi requested a review from Pochomo March 26, 2025 09:56
@sohyunxxi sohyunxxi self-assigned this Mar 26, 2025
@Pochomo
Copy link
Collaborator

Pochomo commented Mar 28, 2025

확인했습니다. 코드 구조가 잘 정리된거같습니다.

@Pochomo Pochomo merged commit c8a8125 into develop Mar 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants