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

[박수민] week14 #464

Conversation

ssumai-kr
Copy link
Collaborator

요구사항

기본

로그인, 회원가입 페이지를 만들어 주세요.
로그인 페이지의 url path는 ‘/signin’, 회원가입 페이지의 url path는 ‘/signup’ 입니다.
Github에 위클리 미션 PR을 만들어 주세요.
React, Next.js를 사용해 진행합니다.

심화

  • [x]
  • []

주요 변경사항

스크린샷

image

멘토에게

너무 늦어서 죄송합니다..!
Next로 프로젝트를 옮기는 것에 시간을 많이 할애 하느라
로그인과 회원가입 페이지는 레이아웃까지만 구현 하였습니다..!

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@ssumai-kr ssumai-kr requested a review from devym-37 May 21, 2024 17:32
@ssumai-kr ssumai-kr self-assigned this May 21, 2024
@ssumai-kr ssumai-kr added 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 미완성🫠 죄송합니다.. labels May 21, 2024
Copy link
Collaborator

@devym-37 devym-37 left a comment

Choose a reason for hiding this comment

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

우선 signin / signup 파일만 확인하였습니다!
다른 부분들은 천천히 작성해주시면 좋을 것 같습니다.
혹시라도 짧게 리뷰해드렸는데, 더 궁금하신 부분이 계시다면 말씀해주시면 좋을 것 같아요!
고생하셨습니다!

);
}

export default Signin;
Copy link
Collaborator

Choose a reason for hiding this comment

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

잘 작성해주셨습니다

<label className={styles.label}>비밀번호 확인</label>
<div className={styles.main}>
<input name="password" type="password" className={styles.checkPassword} required />
<i className="fa fa-eye fa-lg"></i>
Copy link
Collaborator

Choose a reason for hiding this comment

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

children이 없는 태그들은 단일 태그로 작성해주시면 좋을 것 같습니다

);
}

export default Signup;
Copy link
Collaborator

Choose a reason for hiding this comment

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

다른 부분은 잘 작성해주셨습니다

@ssumai-kr ssumai-kr changed the base branch from main to part2-박수민 May 25, 2024 03:08
@devym-37 devym-37 merged commit e4b7a08 into codeit-bootcamp-frontend:part2-박수민 May 29, 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