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.

Sign up for GitHub

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

Jump to bottom

[김혜선] sprint7 #497

Open
wants to merge 7 commits into
base: React-김혜선
Choose a base branch
from
Open

[김혜선] sprint7 #497

wants to merge 7 commits into from

Conversation

kim-hye-sun
Copy link
Collaborator

@kim-hye-sun kim-hye-sun commented Apr 26, 2024 •
edited
Loading

요구사항

기본

  • 각 상품 클릭 시 상품 상세 페이지로 이동합니다.
  • response 로 받은 아래의 데이터로 화면을 구현합니다
  • 목록으로 돌아가기 버튼을 클릭하면 중고마켓 페이지 주소인 “/items” 으로 이동합니다
  • 문의하기에 내용을 입력하면 등록 버튼의 색상은 “3692FF”로 변합니다.
  • response 로 받은 아래의 데이터로 화면을 구현합니다

주요 변경사항

  • 아이템 상세 페이지 작업

스크린샷

screencapture-localhost-3000-product-18-2024-04-27-02_07_50
screencapture-localhost-3000-product-18-2024-04-27-02_08_07

멘토에게

  • 저번에 코드 리뷰로 말씀해주신 부분 수정했습니다.
    다만 custom hook으로 api 호출하는 부분은 말씀 주신대로 수정했더니 orderBy가 좋아요, 최신순으로 나뉘어져 있어 무한 호출해서 그 부분은 빼고 수정했습니다.
  • 시간이 촉박하여 commit을 나누지 못했습니다.

@kim-hye-sun kim-hye-sun changed the title React 김혜선 sprint7 [김혜선] sprint7 Apr 26, 2024
@kim-hye-sun kim-hye-sun requested a review from leejyart April 26, 2024 17:14
@kim-hye-sun kim-hye-sun added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leejyart leejyart Awaiting requested review from leejyart

Assignees
No one assigned
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kim-hye-sun