-
Notifications
You must be signed in to change notification settings - Fork 47
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
전남대 FE_임지환 5주차 과제 #17
base: dlawlghks
Are you sure you want to change the base?
전남대 FE_임지환 5주차 과제 #17
Conversation
멘토님 추가로 STEP2도 진행해보았습니다! |
} | ||
|
||
const storedUsers = JSON.parse(localStorage.getItem('users') || '[]'); | ||
const userExists = storedUsers.some((user: { id: string }) => user.id === id); |
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.
컴포넌트가 렌더링 될 때마다 storedUsers.some 연산을 수행할 텐데요. useMemo
으로 메모이제이션 해주는게 좋을거 같습니다.
useEffect(() => { | ||
const fetchWishlist = async () => { | ||
if (authInfo) { | ||
setLoading(true); |
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.
authInfo
의 유무에 상관없이 loading 상태로 진입하는게 좋을거 같습니다.
setLoading(true); | ||
try { | ||
const response = await fetchInstance.get( | ||
'/api/wishes?page=0&size=10&sort=createdDate,desc', |
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.
'/api/wishes?page=0&size=10&sort=createdDate,desc', | |
'/api/wishes', { params: { page: 0, size: 10, sort: 'createdDate,desc' } } |
안녕하세요 멘토님.
5주체 과제 STEP1을 진행해보았습니다.
열심히 해보았지만, test 코드를 작성하는 것이 조금 미숙하다 보니 어려움이 있었습니다.
Goods/Detail/index.test.tsx 코드에서는 테스트가 잘 되었지만
Order/index.test.tsx 코드를 작성하는 도중 현금영수증 체크박스 상태에 따른 필드 활성화 상태를 확인하는 테스트에서 문제가 생겼습니다.
혹시 이 문제들을 해결하기 위해 제가 놓치고 있는 부분이 있을까요? 추가적으로 확인해봐야 할 사항이 있다면 조언 부탁드립니다.
항상 피드백 해주셔서 감사합니다.