-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feature/main feeds api #38
Conversation
- add feeds API function - add query key management file - add custom hook useFeeds using useQuery - change feed components type
- add feed infinite scroll using useInfiniteQuery and intersectionObserver - combine query key to useQuery
- change common feed type according to API - add feedMutations file
- add ncloud proxy server url - export HttpSuccessType
- link comment like API - link comment dislike API - link comment list API
❌ Deploy Preview for prostargram failed.
|
Quality Gate failedFailed conditions |
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.
고생 많으셨습니다!! 👍
url: `/comments/${commentId}/likes`, | ||
headers: { | ||
Authorization: | ||
'Bearer eyJhbGciOiJIUzUxMiJ9.eyJhdXRob3JpdHkiOiJOT1JNQUxfVVNFUiIsInRva2VuVHlwZSI6IkFDQ0VTU19UT0tFTiIsInN1YiI6IjEiLCJleHAiOjE3NDI3MTUzMDV9.eFVccnbsYRqpJFzg2rL5LNgsMLoEFPESoj3W4e5bgm2Uzy7-DCjA6hgixDZ8MYgXeV6q7VetKQTab-pH-g0caw', |
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.
동일한 테스트용 토큰을 활용하신다면
별도의 파일에 변수로 분리해서 import하면 더 좋을 것 같습니다. 😅
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.
나중에 다 지워질 부분이라 일단 저렇게 냅뒀는데, 리팩토링 시 참고하겠습니다 !!
Pull Request 요약
메인 피드 API 연동 작업
작업 내용
참고 사항
서버측 피드 데이터가 없어 현재 mock.ts 파일에 있는 ALL_FEEDS 데이터를 임시로 넣어둔 상태입니다.
메인 피드에 대한 리팩토링은 서버측에서 피드데이터가 추가되면 별도의 브랜치로 진행 할 예정입니다.
관련 이슈