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/#20 get detail feed detail use case #72

Merged
merged 6 commits into from
May 2, 2024

Conversation

SeongJongHo
Copy link
Collaborator

:: 최근 작업 주제 (하나 이상의 주제를 선택해주세요.)

  • 기능 추가
  • 리뷰 반영
  • 리팩토링
  • 버그 수정
  • 컨벤션 수정

:: 구현 목표 (해당 브랜치에서 구현하고자 하는 하나의 목표를 설정합니다.)

  • 특정 id값의 Feed 불러오는 use_case 작성

:: 구현 사항 설명 (작업한 내용을 상세하게 기록합니다.)

  1. �Feed 데이터 불러오기
  • 파라미터로 받은 id값을 feedRepository에 넘겨주어 Feed 데이터를 받아오는 로직
  1. FeedUseCase test code 작성
  • 작성된 UseCase에 대한 테스트 코드 및 MockFeedRepository 작성

:: 성장 포인트 (해당 기능을 구현하며 고민했던 사항이나 새로 알게된 부분, 어려웠던 점 등을 작성합니다.)

  • Data 레이어가 작성되지 않은 상태로 Domain의 Repository 인터페이스만을 가지고 개발하는게 처음이다 보니 그 부분이 어려웠습니다.

:: 기타 질문 및 특이 사항

없음

@SeongJongHo SeongJongHo added ✨ Feature 기능 개발 [PR] 리뷰요청 리뷰 요청 드립니다. labels May 2, 2024
@SeongJongHo SeongJongHo self-assigned this May 2, 2024
@SeongJongHo SeongJongHo linked an issue May 2, 2024 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@hoogom88 hoogom88 left a comment

Choose a reason for hiding this comment

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

LGTM👍

Copy link
Collaborator

@KimDonghyeok KimDonghyeok left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Collaborator

@gyubit gyubit left a comment

Choose a reason for hiding this comment

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

LGTM👍🏼!

@SeongJongHo SeongJongHo merged commit 23dc768 into dev May 2, 2024
2 checks passed
@SeongJongHo SeongJongHo deleted the feat/#20-get-detail-feed-detail-use-case branch May 2, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 [PR] 리뷰요청 리뷰 요청 드립니다.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[OOTD 상세 화면] GetDetailFeedDetailUseCase 작성
4 participants