Conversation
| tagToRemove => { | ||
| const updatedList = tagList.filter(tag => tag !== tagToRemove); | ||
| tagIdToRemove => { | ||
| const updatedList = tagList.filter(tag => tag.id !== tagIdToRemove); | ||
| setTagList(updatedList); | ||
| onTagListChange(updatedList); |
There was a problem hiding this comment.
p3;
상위 form에서 values에 tagList 상태를 관리하고 있으니,
TagInput이라는 컴포넌트에서도 tagList 상태를 추가로 선언하기 보다는
상위 form에서 전달받은 values.tagList 를 사용하는 건 어떨까요?
setTagList(updatedList)와 onTagListChange(updatedList)
이렇게 똑같은 updatedList를 두 번 업데이트하는건 조금 비효율적인 것 같아요!
There was a problem hiding this comment.
오류가 나서... 줄이는게 안되네요.
| tagToRemove => { | ||
| const updatedList = tagList.filter(tag => tag !== tagToRemove); | ||
| tagIdToRemove => { | ||
| const updatedList = tagList.filter(tag => tag.id !== tagIdToRemove); |
There was a problem hiding this comment.
p5;
요런 경우에는 기존 상태를 참조하기보다는 useState의 set함수에서 사용할 수 있는
이전 상태의 스냅샷을 사용하여 상태를 업데이트 하는 것도 괜찮을 것 같아요!
setTagList(prevTagList => {
const updatedList = prevTagList.filter(tag => tag.id !== tagIdToRemove);
return updatedList
})
|
윤주님 고생하셨어요! api 폴더는 현재도 괜찮은데요! 추후 products가 아닌 리소스에 대한 요청 등을 고려해 아래와 같은 느낌으로 디렉토리를 구성해도 괜찮을 것 같아요! /services
-/api
-/products
/index.js
-/users // 예시
/index.js +) 일부 파일에서 confilct이 발생하고 있는데, 이 부분 해결해서 올려주시면 머지해드릴게요! |
수정 권한이 없는지... 뭔가 잘 안되네요..ㅠㅠ 이번 내용 적용해서 다시 한번 올려볼께요 |
스프린트 7미션
요구사항
기본
상품 상세
=> favoriteCount : 하트 개수
=> images : 상품 이미지
=> tags : 상품태그
=> name : 상품 이름
=> description : 상품 설명
상품 문의 댓글
=> image : 작성자 이미지
=> nickname : 작성자 닉네임
=> content : 작성자가 남긴 문구
=> description : 상품 설명
=> updatedAt : 문의글 마지막 업데이트 시간
스크린샷
멘토에게