-
Notifications
You must be signed in to change notification settings - Fork 0
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] 셰어 공개상담 개발 요청 사항 반영 / react-query 반영 #300
Conversation
✅ Deploy Preview for candid-semolina-d0db42 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
고생하셨습니다~!
queryKey: ['infiniteGetPostsCutsomersResponse', isChecked], | ||
queryFn: async ({ pageParam }) => | ||
await getPostsCutsomers(pageParam).then((res) => { | ||
console.log(res.data); |
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.
console.log
는 삭제해주시면 좋을 거 같습니다!
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.
반영하였습니다! b6edd86
} | ||
}; | ||
export const axiosGet = async (url: string, params?: any) => | ||
await instance.get(url, { params }); |
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.
try-catch문과 이별하기 좋아요 ㅎㅎ
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.
이 방식이 더 깔끔한 거 같습니다~ Goood
const { | ||
data: openConsults, | ||
isLoading, | ||
fetchNextPage, | ||
hasNextPage, | ||
isFetching, | ||
isFetchingNextPage, | ||
isLoadingError, | ||
} = useInfiniteQuery<GetPostsCutsomersResponse[]>({ |
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.
공개 상담 단건 조회 페이지를 비롯한 상담 상세 페이지에서 게시물 좋아요
, 북마크
, 댓글 좋아요
혹은 아이콘을 눌렀을 때 숫자 부분 width가 달라져서 아이콘이 밀리는 거 같습니다!
좋아요/스크랩 숫자 부분을 좋아요 자릿수에 따라 width를 지정해두는 것도 방법일 거 같습니다!
2024-06-14.7.39.58.mov
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.
앗 그런 이슈가 있군요!
저는 layout을 구성할 때 width, height의 직접적인 선언을 지양하는 것이 반응형에 좋은 practice라고 생각하여 위와 같이 구현하였습니다!
여러 부분에서 사용하는 태그이기 때문에 다음 작업에서 따로 마이그레이션하는 방향으로 하겠습니다!
Checklist
Description
Related Issues
#294
To Reveiwer