Skip to content

Conversation

@qowldud
Copy link

@qowldud qowldud commented May 8, 2025

✏️ 작업 내용

Chapter 06 - Tanstack Query 입문 1편: useQuery와 useInfiniteQuery로 무한 스크롤과 효율적인 데이터 패칭하기

#️⃣ 연관된 이슈

#31


📷 작업 결과

너무 커서 안올라갑니다..ㅠㅠㅠㅠㅠㅠ노션으로 확인해주세요..


💡 함께 공유하고 싶은 부분

  • 무한스크롤부분은 아직 어려운것같아요... 그리고 스켈레톤도 했는데 데이터를 빨리 받아와서 그런지 잘 보이지는 않습니다..!! 무한스크롤이랑 tanstack query공부는 더 해봐야할것같아요

🤔 질문

해당 주차 워크북을 공부하면서 궁금했던 질문들을 남겨주세요.


✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하였나요?
  • 실습/미션을 수행하였나요?

✅ 컨벤션 체크리스트

  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 코드리뷰를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있나요?

Choose a reason for hiding this comment

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

LPDetail 컴포넌트들 따로 정리하니까 훨씬 깔끔한 거 같아용!

Copy link

@namgungseok12 namgungseok12 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

@namgungseok12 namgungseok12 left a comment

Choose a reason for hiding this comment

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

good good

Copy link
Contributor

@duwlsssss duwlsssss 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
Contributor

@duwlsssss duwlsssss left a comment

Choose a reason for hiding this comment

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

고생하셨습니닷

>
{!data || isLoading
? Array.from({ length: 10 }).map((_, i) => (
<CommentSkeleton key={i} />
Copy link
Contributor

Choose a reason for hiding this comment

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

skeleton 컴포넌트에는 개수만 넘기고 안에서 배열로 처리해도 깔끔할 것 같아요!

Copy link
Author

Choose a reason for hiding this comment

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

오 넵넵 반영해보겠습니다!

@qowldud qowldud merged commit f0ab8e6 into judy/main May 12, 2025
1 check passed
@qowldud qowldud deleted the judy/#31 branch May 13, 2025 05:33
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.

5 participants