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(home): 홈 페이지의 인기 아티클 태그 목록 실제 데이터를 조회 후 연동하도록 합니다. #112

Merged
merged 4 commits into from
Sep 17, 2023

Conversation

innocarpe
Copy link
Collaborator

📌 이슈 링크


📖 작업 배경

  • '인기 아티클 태그 목록 조회'를 위한 서비스와 useQuery 를 작성합니다.
  • 홈 피드 컨텐츠 쪽에서 useQuery 를 두 개 사용해야 해서, 아티클 프리뷰 쪽과 인기 아티클 태그 목록의 useQuery 의 반환값에서 좀 더 구체적인 네이밍을 사용하도록 합니다.
  • 홈 화면에서 실제 데이터를 불러와 연동하도록 합니다.

🛠️ 구현 내용

  • ㅇㅇPagination 에서 액션이 발생할 때 상위로 이벤트를 전달하도록 구현합니다.
  • 홈 피드 컴포넌트에서 페이지 변경이 있을 때 새로 아티클 목록을 받아오도록 로직을 개선합니다.
  • 사이드바를 아티클 로딩에 따라 노출 처리를 묶어뒀었는데, 향후 더 좋은 방향으로 개선하기 위해 로딩 상태와 엮지 않도록 변경했습니다.

💡 참고사항


🖼️ 스크린샷

Screen.Recording.2023-09-14.at.11.12.27.PM.mov

const { articlePreviews, isArticlePreviewsLoading } =
useArticlePreviewQuery(currentPageIndex);
const { popularArticleTags, isPopularArticleTagsLoading } =
usePopularArticleTagsQuery();
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이 PR 부터 두 개의 useQuery 를 사용하게 되어서, 네이밍을 좀 더 구체화하는게 가독성을 높힐 수 있다고 생각했습니다.

@innocarpe innocarpe added feature A new feature status: dependent 이전 PR의 커밋이 포함되어 있어 대기해야 함 labels Sep 14, 2023
Base automatically changed from carpe/home-pagination-click to team6/innocarpe September 17, 2023 08:17
@innocarpe innocarpe removed the status: dependent 이전 PR의 커밋이 포함되어 있어 대기해야 함 label Sep 17, 2023
@innocarpe innocarpe merged commit 36b3d97 into team6/innocarpe Sep 17, 2023
@innocarpe innocarpe deleted the carpe/home-popular-tags branch September 17, 2023 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant