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

레시피 리스트 fetch 데이터 관련 로직 수정 #37

Merged
merged 4 commits into from
Jan 8, 2025

Conversation

GeonH0
Copy link
Collaborator

@GeonH0 GeonH0 commented Jan 8, 2025

PR 개요

레시피 리스트 fetch 데이터 관련 로직 수정


주요 변경 사항

  1. 데이터 네이밍과 형식 수정
    • recipeImageUrl의 형식을 배열에서 단일 URL로 변경했습니다
    • 네트워크 통신에서 받아오는 Recipe 데이터를 DTO에 맞게 네이밍 및 형식 정리했습니다

  2. 레시피 리스트 fetch 시 필요한 파라미터 추가 및 선언
    • currentPage, targetPage, boundaryID를 네트워크 요청에 추가하여 효율적으로 데이터를 가져오도록 수정했습니다
    • 새로운 파라미터를 처리하기 위해 FetchFeedListUseCase 및 관련 메서드에 수정 반영했습니다..

  3. 레시피 리스트 fetch 로직 수정
    • boundaryID 값이 현재 페이지의 최대 recipeId를 기준으로 설정되도록 구현했습니다.

  4. Repository의 파라미터 수정
    • 레시피 데이터를 불러오는 Repository 계층에서 새로운 네트워크 요청 형식에 맞게 파라미터 변경.
    • 기존 pageNumber 단일 값 요청을 currentPage, targetPage, boundaryID로 대체.


Copy link

sonarqubecloud bot commented Jan 8, 2025

@GeonH0 GeonH0 merged commit 79561cc into main Jan 8, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant