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

Conversation

han9898
Copy link
Collaborator

@han9898 han9898 commented Jun 7, 2024

요구사항

기본

상품 등록

  • 상품 등록 페이지 주소는 “/additem” 입니다.
  • 페이지 주소가 “/additem” 일때 상단네비게이션바의 '중고마켓' 버튼의 색상은 “3692FF”입니다.
  • 상품 이미지는 최대 한개 업로드가 가능합니다.
  • 각 input의 placeholder 값을 정확히 입력해주세요.
  • 이미지를 제외하고 input 에 모든 값을 입력하면 ‘등록' 버튼이 활성화 됩니다.
  • API를 통한 상품 등록은 추후 미션에서 적용합니다.

심화

상품 등록

  • 이미지 안의 X 버튼을 누르면 이미지가 삭제됩니다.
  • 추가된 태그 안의 X 버튼을 누르면 해당 태그는 삭제됩니다.

주요 변경사항

스크린샷

localhost_3000_additem (1)

멘토에게

  • 레이아웃 작업을 잠시 건너뛰고 기능에 집중을 하려고 했으나 눈에 보여서 레이아웃에도 집중을 했습니다... 다음 미션에서 api 도전해보도록 하겠습니다!
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@han9898 han9898 requested a review from kich555 June 7, 2024 14:20
@han9898 han9898 self-assigned this Jun 7, 2024
@han9898 han9898 added 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 죄송합니다.. labels Jun 7, 2024
Copy link
Collaborator

@kich555 kich555 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ㅎㅎ

page,
pageSize,
order,
});
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍👍👍

const [productName, setProductName] = useState("");
const [productDescription, setProductDescription] = useState("");
const [price, setPrice] = useState("");
const [tags, setTags] = useState("");
Copy link
Collaborator

Choose a reason for hiding this comment

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

form 내부 input의 값들은 하나의 객체 상태로 관리하는게 좋을것 같아요 ㅎ
상태는 최대한 줄이면 줄일수록 좋습니다 ㅎ

disabled={!isFormValid}
>
등록
</button>
Copy link
Collaborator

Choose a reason for hiding this comment

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

form 내부의 등록 버튼같은 경우 type=submit을 꼭 붙혀주세요 ㅎ

@kich555 kich555 merged commit 756a0e7 into codeit-bootcamp-frontend:React-장소희 Jun 10, 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