경북대 FE_이효은 5주차 과제 Step2 #49
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sjoleee
approved these changes
Jul 29, 2024
Comment on lines
79
to
87
| {hasNextPage && ( | ||
| <VisibilityLoader | ||
| callback={() => { | ||
| if (!isFetchingNextPage) { | ||
| fetchNextPage(); | ||
| } | ||
| }} | ||
| /> | ||
| )} |
There was a problem hiding this comment.
계속 재요청이 발생하는 버그는 요 부분 때문이에요!
VisibilityLoader가 화면에 계속 보이면서 지속적으로 fetchNextPage가 호출되고 있어요.
VisibilityLoader는 화면에 보이면 콜백을 호출하는 역할로 보이는데... 제가 생각하기엔 VisibilityLoader를 사용하지 않는 편이 나을 것 같아요.
로더가 화면에 보인다는건 다음 페이지를 불러오는 중이라는 의미인데, 다음 페이지를 불러오는 중이라면 아무런 동작을 하지 않아야 합니다.
다음 페이지를 불러오는 중이 아니라면 로더가 화면에 보이면 안되구요.
해당 컴포넌트의 구현이 조금 이상한 것 같다는 느낌입니다 😅
Comment on lines
59
to
77
| {wishList.map(({ id, product }) => ( | ||
| <div | ||
| key={id} | ||
| css={css` | ||
| display: flex; | ||
| flex-direction: column; | ||
| `} | ||
| > | ||
| <DefaultGoodsItems | ||
| imageSrc={product.imageUrl} | ||
| title={product.name} | ||
| amount={product.price} | ||
| subtitle={''} | ||
| /> | ||
| <Button theme="darkGray" size="small" onClick={() => handleDeleteWish(product.id)}> | ||
| 삭제 | ||
| </Button> | ||
| </div> | ||
| ))} |
There was a problem hiding this comment.
useGetWishList가 반환한 위시리스트 data를 사용하지 않고 비어있는 wishList 상태를 사용해서 렌더링하고 있네요!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요 멘토님,,
구현을 하긴 했는데 mock 파일 작성하는 부뷴이,, 이렇게 짜는게 맞는지 모르겠습니다.!!!
그리고 myaccount 페이지에 들어가면 wishlist 목록이 무한로딩되는데 이유를 모르겠습니다ㅠㅠ
감사합니다!