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 #206

Conversation

KorpoQ
Copy link
Collaborator

@KorpoQ KorpoQ commented Jul 4, 2024

체크리스트 [기본]

상품 상세

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

상품 문의 댓글

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

멘토님께

  • 어떻게든 팀 프로젝트 전에...

@KorpoQ KorpoQ added the 미완성🫠 죄송합니다.. label Jul 4, 2024
@KorpoQ KorpoQ requested a review from jyh0521 July 5, 2024 18:08
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.

과제하느라 고생하셨습니다!

public/index.html Show resolved Hide resolved
src/App.js Show resolved Hide resolved
Comment on lines +53 to +59
Button.propTypes = {
size: PropTypes.oneOf(["small", "medium", "large"]).isRequired,
height: PropTypes.oneOf(["42", "48"]),
theme: PropTypes.oneOf(["blue", "white"]),
innerText: PropTypes.string,
to: PropTypes.string,
};
Copy link
Collaborator

Choose a reason for hiding this comment

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

propTypes를 사용해보셨군요! 좋네요.

src/components/UI/jsx/DropDownList.jsx Show resolved Hide resolved
src/components/UI/jsx/Input.jsx Show resolved Hide resolved
src/components/UI/scss/Button.scss Show resolved Hide resolved
@@ -1,6 +1,6 @@
.searchBar {
display: flex;
background-color: #f3f4f6;
background-color: var(--gray100);
Copy link
Collaborator

Choose a reason for hiding this comment

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

컬러변수 반영해주신 점 좋네요~

@@ -29,19 +31,45 @@ function AddItemPage() {
});
};

const handleImageDelete = (e) => {
// 이미지 삭제 버튼 클릭 시,
Copy link
Collaborator

Choose a reason for hiding this comment

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

함수명을 구체적으로 잘 작성해주셔서 주석은 달아주시지 않아도 어떤 역할을 하는 함수인지 잘 알아볼 수 있을 것 같습니다.

Comment on lines +13 to +14
const ORDER_BY_RECENT = "recent";
const ORDER_BY_FAVORITE = "favorite";
Copy link
Collaborator

Choose a reason for hiding this comment

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

상수 관리해주신 점 좋네요!

@jyh0521 jyh0521 merged commit 14a68f7 into codeit-bootcamp-frontend:React-임귀태 Jul 8, 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