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/#144] 카드리스트 수정 및 관련 API 연결 #169

Merged
merged 11 commits into from
Jan 22, 2025

Conversation

chaeneey
Copy link
Collaborator

📌 관련 이슈번호


체크리스트

  • 🎋 base 브랜치를 develop 브랜치로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏗️ 빌드는 성공했나요? (yarn build)
  • 🧹 불필요한 코드는 제거했나요? e.g. console.log
  • 🙇‍♂️ 리뷰어를 지정했나요?
  • 🏷️ 라벨은 등록했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. CardList 및 useFilteredCardList 수정

    • CardList: CardList 데이터 받는 부분 타입 가드를 추가했습니다. Store 데이터인지 Design 데이터 인지 확인하는 과정입니다! 또한, likedStoreDesign 찜한 스토어의 디자인을 보는 부분 item이 하나 더 있어서, 그 부분 추가했습니다!
    • useFilteredCardList: 최신순, 인기순 option 상태와 함수를 관리하는 훅으로 변경했습니다. 기존에는 해당 훅에서 api 요청 및 데이터를 가져오는 것까지 진행하려고 했으나, CardList가 12개의 api를 대응해야 하다보니, 수많은 타입 오류 등의 문제로, api는 각자 사용하는 페이지 내에서 불러오는 형태로 변경했습니다.
  2. 디자인 둘러보기 조회 (최신순, 인기순) API 연결 (둘러보기 페이지)

  • useFetchDesignList 만들어서 DesignList 페이지에서 연결했습니다.
  1. 디자인 상세보기 조회 API 연결 (둘러보기 페이지)
  • useFetchDesignDetail 만들어서 DesignSearchModal 컴포넌트에서 연결했습니다.
  1. 찜한 스토어 조회 (마이 페이지)
  • useFetchLikedStoreList 만들어서 My 페이지에서 연결했습니다.
  1. 인기 스토어 랭킹 조회 (메인 페이지)
  • useFetchStoreRank 만들어서 Home 페이지에서 연결했습니다.

📢 To Reviewers


📸 스크린샷 or 실행영상

Copy link
Collaborator

@zzz-myam zzz-myam left a comment

Choose a reason for hiding this comment

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

정말 고생하셨습니다! cardlist를 만들면서 얼마나 고생했는지 알기에 더 마음이 찡한 PR이네요! 고민한 시간만큼 듬뿍 성장하셨을 것 같아서 제가 다 뿌듯합니다. 천재 개발자의 남은 3일도 응원합니다!

Copy link
Collaborator

@thisishwarang thisishwarang left a comment

Choose a reason for hiding this comment

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

정말 어렵고 고생이 많았던 CardList 컴포넌트와 useFilteredCardList 고생 많으셨습니다!

Copy link
Collaborator

@youtheyeon youtheyeon left a comment

Choose a reason for hiding this comment

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

애증의 카드리스트.. 수정하느라 너무 고생 많으셨습니다 🥹
미안하고 감사합니다.........🥺💛

Comment on lines +9 to +20
interface cakeType {
cakeId: number;
isLiked: boolean;
imageUrl: string;
}

interface DesignDetailResponse {
storeId: number;
storeName: string;
station: string;
cake: cakeType[];
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

types 폴더로 빼면 좋을 것 같습니다!

Comment on lines +11 to +12


Copy link
Collaborator

Choose a reason for hiding this comment

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

?

Comment on lines +40 to +48
if (isLoading) {
return <div>로딩중...</div>;
}

if (!data) {
return <div>데이터 없음...</div>;
}

console.log(cakeId);
Copy link
Collaborator

Choose a reason for hiding this comment

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

추후 삭제 부탁드립니다 🙏

@thisishwarang thisishwarang merged commit a0650a6 into develop Jan 22, 2025
1 check passed
@thisishwarang thisishwarang deleted the feat/#144/design-list-page-api branch January 22, 2025 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 디자인 둘러보기 조회 API 연결
4 participants