-
Notifications
You must be signed in to change notification settings - Fork 2
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
[#56] Feat: 좋아요한 짤 api 및 커스텀 훅 구현 #59
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
src/routes/my-liked-zzal/index.tsx
Outdated
<div className="mb-4 inline-block break-inside-avoid"> | ||
<ZzalCard src={path} alt="" /> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분 key가 빠진것 같습니다 !! ㅎㅎ
import { GetMyLikedZzalsResponse } from "@/types/zzal.dto"; | ||
import http from "./core"; | ||
|
||
const SIZE = 10; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분 따로 Constants 폴더에서 관리하지 않아도 될까요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 아직 확정이 아니라서 우선 여기에 뒀는데 확정되면 옮기려구요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다 재민님!
LGTM입니다 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM~~!🎉🎉🎉🎉
📝 작업 내용
좋아요한 짤 api와 커스텀 훅 구현
useSuspenseInfiniteQuery
를 사용하여 커스텀 훅 구현뷰포트 크기에 맞게 열 개수를 조절
ZzalCard 수정
이미지 크기 이상의 높이를 가져서
h-fit
추가pending 컴포넌트 선언
suspense 사용으로 인해 pending 컴포넌트가 필요해 우선 선언만 해두었습니다.
close #56