Skip to content
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

[나윤주] sprint7 #231

Conversation

naynara87
Copy link
Collaborator

스프린트 7미션

요구사항

  • Github에 PR(Pull Request)을 만들어서 미션을 제출합니다.
  • 피그마 디자인에 맞게 페이지를 만들어 주세요.
  • React를 사용합니다

기본

상품 상세

  • response 로 받은 아래의 데이터로 화면을 구현합니다.
  • 상품 상세 페이지 주소는 “/items/{productId}” 입니다.
    => favoriteCount : 하트 개수
    => images : 상품 이미지
    => tags : 상품태그
    => name : 상품 이름
    => description : 상품 설명
  • 목록으로 돌아가기 버튼을 클릭하면 중고마켓 페이지 주소인 “/items” 으로 이동합니다

상품 문의 댓글

  • response 로 받은 아래의 데이터로 화면을 구현합니다
  • 문의하기에 내용을 입력하면 등록 버튼의 색상은 “3692FF”로 변합니다.
    => image : 작성자 이미지
    => nickname : 작성자 닉네임
    => content : 작성자가 남긴 문구
    => description : 상품 설명
    => updatedAt : 문의글 마지막 업데이트 시간

스크린샷

스크린샷 2024-07-04 오후 9 57 17

멘토에게

  • 컴포넌트 세분화 / useHook 연습 했습니다. 더 단순화하려고 했는데.. 다음에 만들면서 응용해보도록 하겠습니다.

@naynara87 naynara87 requested a review from Taero-Kim July 8, 2024 16:00
@naynara87 naynara87 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Jul 8, 2024
try {
const products = await getProducts({ orderBy, page, pageSize });
setProducts(products.list);
setTotalPageNum(Math.ceil(products.totalCount / pageSize));
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p4;
totalPageNum은 products와 pageSize를 deps로 가지는 useMemo로 관리해도 좋을 것 같아요!

return product.trim() !== '' && content.trim() !== '' && price !== 0 && tag.length > 0;
}, [values]);

const handleChange = useCallback((name, value) => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p4;
handleChangeFormValues 등의 함수명이면 더 직관적일 것 같아요!

@Taero-Kim
Copy link
Collaborator

요정도만 해도 잘하셨어요! 앞으로 프로젝트에서 요렇게 연습하셨던게 도움이 많이 될거에요!

hook도 잘 작성하셨어요!
한 가지 커스텀훅이 너무 많은 일을 담당하지 않도록, 조금 더 고민하고 코드를 작성하시면
더욱 더 좋은 구조의 코드를 작성하실 수 있을 것 같아요👍

@Taero-Kim Taero-Kim merged commit 9fd52e5 into codeit-bootcamp-frontend:React-나윤주 Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants