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

[민지영] sprint6 #110

Conversation

Obsessive-Curiosity
Copy link
Collaborator

요구사항

기본

  • [x]
  • []
  • []

심화

  • [x]
  • []

주요 변경사항

스크린샷

image

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

회원가입 및 로그인 form 버튼
Input component 수정중 (pw 미리보기 토글기능 추가예정)
SNS 로그인 및 회원가입 컴포넌트
ui 수정
create modal component in auth page
<li
key={navItem}
onClick={() => nav(NAV_ITEM[navItem].PATH)}
className="text-gray-600 text-lg font-bold leading-[26px] cursor-pointer md:text-base"
Copy link
Collaborator

Choose a reason for hiding this comment

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

모바일 일때의 폰트사이즈 16px 입니다. text-[16px] md:text-lg 로 적용하셔야 맞을 것 같네요. 반응형에서 보통 모바일의 폰트사이즈는 데스크탑의 사이즈보다 작은 경우가 대부분입니다.

});
}, [data]);

if (error) return <p>데이터 가져오기 실패: {error}</p>; // 오류 발생 시 500 페이지로 이동 (수정 예정)
Copy link
Collaborator

Choose a reason for hiding this comment

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

추후 500 페이지로 이동하도록 수정하신다고 하셨는데, 에러바운더리( https://www.npmjs.com/package/react-error-boundary) 사용하셔서 throw new Error() 로 에러 던지면 에러바운더리에서 받아서 500페이지를 표현하도록하고 사용자가 reset 버튼 눌렀을 때 다시 시도할 수 있게 해주시면 더 좋을 것 같습니다.

@rjc1704
Copy link
Collaborator

rjc1704 commented Jan 28, 2025

대부분의 프론트엔드 요구사항들을 잘 만족시켜주셨습니다. 추가적인 코멘트는 아래에 정리해 드리겠습니다.

  • 상품등록 시 api 요청을 보내지 않고 상태변경까지만 적용하셨군요. 상품등록 post 요청으로 DB 에 데이터 추가 후 상세페이지로 이동하면 상세페이지에서 DB에서 해당 상품id에 해당하는 데이터를 api 요청으로 불러와 렌더링하도록 설정해주셔야 합니다.
  • 페이지 주소가 "/items" 일 때 상단내비게이션바의 "중고마켓" 버튼의 색상은 "3692FF"입니다. 이 부분은 적용이 안되어있군요.
  • 백엔드 작업한 서버 url 로 api 연결하지 않으셨군요. 백엔드 작업하신 것 render.com 에 배포한 url 로 변경해서 api 연결해보시면 좋겠습니다. 백엔드 서버에서 우선 cors 는 오픈하기위해 app.use(cors()) 로 설정해서 테스트하시면 됩니다.

@rjc1704 rjc1704 merged commit bed603a into codeit-sprint-fullstack:react-민지영 Jan 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants