-
Notifications
You must be signed in to change notification settings - Fork 20
[김민식]sprint 6 #73
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
base: react-김민식
Are you sure you want to change the base?
[김민식]sprint 6 #73
The head ref may contain hidden characters: "react-\uAE40\uBBFC\uC2DD"
Conversation
Hoooooou-Jun
left a comment
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.
민식님 안녕하세요. 코드리뷰 완료되었습니다.
전반적으로 말씀하셨다시피 AI의 도움을 받아 작성한 부분이 보이지만, 괜찮습니다. 그럼에도 필요한 부분은 구현 잘 해주셨습니다.
하지만 이번 미션에서 제일 중요한 API 연동은 실제 현업에서도 주가 되는 업무이기에 이 부분은 특히 다시 공부하셔서 확실하게 연습하고 넘어가는 것을 권유 드립니다.
그 외 프론트엔드는 모듈화만 조금 신경써주시면 될 것 같습니다!
리액트는 지금까지 한 내용 정도에서 최대한 익숙해지시려고, 친해지시려고 노력하시는 게 중요할 것 같습니다. 공부를 더 한다기 보다 코드를 몇줄 더 따라치고 연습하는 것이 매우 중요합니다.
State 작성 관련해서는 멘토링때 같이 나누면 좋을 것 같아 이후에 다시 질문 부탁드리겠습니다.
고생 많으셨습니다~!
|
|
||
| try { | ||
| const response = await fetch( | ||
| ` https://panda-market-api.vercel.app/docs/products?${query}` |
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.
민식님께서 만든 백엔드랑 연결하시려면 따로 이 주소를 수정해주시면 됩니다. 지금은 미완성이라 이렇게 두신 것 같네요
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.
더불어 https://panda-market-api.vercel.app/ 이 주소 혹은 API를 쏘게 될 주소는 따로 분리가 가능합니다!
그래서 보통 실제 프로덕션 배포 환경과 개발서버 배포 환경 / 로컬 배포 환경 등 환경에 따라서 이 API 주소가 바뀔 수 있어서 해당 부분은 분리해서 한 곳에서 도메인을 관리해 사용하시는 것이 매우 유용합니다!
| </main> | ||
|
|
||
| <footer> | ||
| <div id="copyright">©codeit - 2024</div> |
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.
해당 footer 부분은 계속해서 반복되는 코드이므로 모듈화 부탁드립니다.
| </nav> | ||
| </div> | ||
| <button type="button" className="loginLink button"> | ||
| <Link to="/Login" aria-label="로그인 페이지로 이동"> |
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.
다른 라우트는 소문자인데 통일 부탁드립니다.
| <div className="feature-content"> | ||
| <h2>Hot item</h2> | ||
| <main className="withHeader"> | ||
| <section id="hero" className="banner"> |
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.
withHeader와 wrapper 조합은 layout 컴포넌트로 분리하면 더 좋습니다.
요구사항
기본
###랜딩 페이지
[ ] HTML과 CSS로 구현한 랜딩페이지를 React로 마이그레이션하세요.
[ ] 랜딩 페이지 url path는 "/"로 설정하세요.
중고마켓 페이지
[ ] 중고마켓 페이지 url path를 "/items"으로 설정하세요.
[ ] 페이지 주소가 "/items" 일 때 상단내비게이션바의 "중고마켓" 버튼의 색상은 "3692FF"입니다.
[ ] 중고마켓 페이지 판매 중인 상품은 본인이 만든 GET 메서드를 사용해 주세요.
[ ] 다만 좋아요 순 정렬 기능은 제외해 주세요.
[ ] 사진은 디폴트 이미지로 프론트엔드에서 처리해주세요.
[ ] 베스트 상품 목록 조회는 구현하지 않습니다.
[ ] '상품 등록하기' 버튼을 누르면 "/registration" 로 이동합니다. ( 빈 페이지 )
상품 등록 페이지
[ ] PC, Tablet, Mobile 디자인에 해당하는 상품 등록 페이지를 만들어 주세요.
[ ] 상품 등록 url path는 "/registration"입니다.
[ ] 상품 등록은 본인이 만든 POST 메서드를 사용해 주세요.
[ ] 등록 성공 시, 해당 상품 상세 페이지로 이동합니다. (빈페이지)
심화
주요 변경사항
스크린샷
멘토에게