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

[장혁수] sprint5 #144

Conversation

hyeoksuJ
Copy link
Collaborator

@hyeoksuJ hyeoksuJ commented Jun 21, 2024

요구사항

기본

중고마켓

  • 중고마켓 페이지 주소는 “/items” 입니다.
  • 페이지 주소가 “/items” 일때 상단네비게이션바의 '중고마켓' 버튼의 색상은 “3692FF”입니다.
  • 상단 네비게이션 바는 이전 미션에서 구현한 랜딩 페이지와 동일한 스타일로 만들어 주세요.
  • 상품 데이터 정보는 https://panda-market-api.vercel.app/docs/#/ 에 명세된 GET 메소드 “/products” 를 사용해주세요.
  • '상품 등록하기' 버튼을 누르면 “/additem” 로 이동합니다. ( 빈 페이지 )
  • 전체 상품에서 드롭 다운으로 “최신 순” 또는 “좋아요 순”을 선택해서 정렬을 할 수 있습니다.

중고마켓 반응형

  • 베스트 상품
    Desktop : 4개 보이기
    Tablet : 2개 보이기
    Mobile : 1개 보이기
  • 전체 상품
    Desktop : 12개 보이기
    Tablet : 6개 보이기
    Mobile : 4개 보이기

심화

  • 페이지 네이션 기능을 구현합니다.

주요 변경사항

스크린샷

멘토에게

  • 아직 미완성입니다.
  • 리액트를 활용하는 게 아직 감이 안 잡혀서 pr 먼저 올리고 계속 진행하겠습니다.

@hyeoksuJ hyeoksuJ requested a review from jyh0521 June 21, 2024 10:04
@hyeoksuJ hyeoksuJ added the 미완성🫠 죄송합니다.. label Jun 21, 2024
Copy link
Collaborator

@jyh0521 jyh0521 left a comment

Choose a reason for hiding this comment

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

과제하느라 고생하셨습니다! 계속 발전시켜주시면 좋을 것 같아요 ㅎㅎ

@@ -0,0 +1,10 @@
<!DOCTYPE html>
<html lang="en">
Copy link
Collaborator

Choose a reason for hiding this comment

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

lang ko로 바꿔주시면 좋을 것 같아요!

<html lang="en">
<head>
<meta charset="utf-8" />
<title>React App</title>
Copy link
Collaborator

Choose a reason for hiding this comment

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

저희 판다마켓이니까 title도 변경해주시면 될 것 같습니다.

src/App.js Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

저번에 말씀드린대로 App.js에다가 헤더 잘 넣어주셨네요! Items는 라우터 훅을 사용해서 원하는 경로일때만 띄우게 해주시면 좋을 것 같습니다.

src/index.css Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

초기화 목적인 css라면, index.html에 link 태그 혹은 App.js에서 import로 불러오게 해주시면 됩니다!

@jyh0521 jyh0521 changed the base branch from main to React-장혁수 June 22, 2024 01:51
@jyh0521 jyh0521 merged commit e781f2e into codeit-bootcamp-frontend:React-장혁수 Jun 22, 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.

4 participants