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

Conversation

Kimyu94
Copy link
Collaborator

@Kimyu94 Kimyu94 commented Jun 7, 2024

요구사항

기본

  • [x]
  • []
  • []

심화

  • [x]
  • []

주요 변경사항

스크린샷

멘토에게

  • 현재 스프린트 5,6 작업중입니다.
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

hanseulhee and others added 26 commits October 10, 2023 14:15
…ithub-actions

[Fix] delete merged branch github action
@Kimyu94 Kimyu94 requested a review from airman5573 June 7, 2024 07:43
@Kimyu94 Kimyu94 added 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 죄송합니다.. labels Jun 7, 2024
Copy link
Collaborator

@airman5573 airman5573 left a comment

Choose a reason for hiding this comment

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

안녕하세요 유정님, 미션 진행하느라 수고하셨습니다.

UI

image

파일구조

image

코드

상세한 코드 리뷰는 코멘트를 확인해 주세요 :D

@@ -0,0 +1,23 @@
module.exports = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

eslint를 설정하셨군요 👍 👍


function Button({ children }) {
return (
<>
Copy link
Collaborator

Choose a reason for hiding this comment

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

[수정해주세요]

여기서 Fragment는 지우셔도 됩니다

import Profile from "../assets/images/profile.png";
import "../styles/Header.css";
import { Link, NavLink } from 'react-router-dom';
import Logo from '@/assets/images/logo.png';
Copy link
Collaborator

Choose a reason for hiding this comment

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

absolute path를 사용하셨군요 👍

}`;

return (
<>
Copy link
Collaborator

Choose a reason for hiding this comment

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

[수정해주세요]

여기서 Fragment는 없어도 됩니다!

name="item_name"
placeholder="상품명을 입력해주세요"
/>
<div className={styles.label}>상품 이미지</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

[참고해주세요]

여기서는 div대신 label이 더 잘어울립니다!
그래야 '상품 이미지'를 클릭했을때 input에 focus가 생깁니다.

@airman5573 airman5573 merged commit 142d95b into codeit-bootcamp-frontend:React-김유정 Jun 17, 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.

5 participants