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

Feat: create login page #45

Merged
merged 6 commits into from
Jan 8, 2024
Merged

Feat: create login page #45

merged 6 commits into from
Jan 8, 2024

Conversation

NamgungJongMin
Copy link
Member

개요

로그인 페이지 생성

스크린샷

image

주요 내용

  • 로그인 form 컴포넌트 분리
  • 각 input section 별 focus시 값을 가지고 있다면 remove button 생성
  • api 요청 전 클라이언트 자체 validation
  • 타입 분리

고민한 점, 질문거리

react hook form에서 onsubmit 모드로 submit 마다 validation 결과를 반영하고 싶었으나 설정한 pattern의 validation을 통과하지 않으면 submit 기능 자체를 막아놓아서 submit으로 validation 결과를 렌더링 할 수 없음.
일단 submit 대신 onclick으로 이벤트 걸고 onchange모드로 변경. 추가적으로 확인할 예정

close #19

@NamgungJongMin NamgungJongMin linked an issue Jan 8, 2024 that may be closed by this pull request
3 tasks
Copy link

vercel bot commented Jan 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
trip-vote ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 8, 2024 4:36am

@NamgungJongMin NamgungJongMin self-assigned this Jan 8, 2024
@NamgungJongMin NamgungJongMin added this to the TripVote FE development milestone Jan 8, 2024
Copy link
Contributor

@JeongMin83 JeongMin83 left a comment

Choose a reason for hiding this comment

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

선 어프루브 후 읽기 하겠습니다

@NamgungJongMin
Copy link
Member Author

선 어프루브 후 읽기 하겠습니다

코멘트 다신건 이후 이슈 올리고 바로 적용하겠슴다~

@NamgungJongMin NamgungJongMin merged commit fd29d7b into dev Jan 8, 2024
3 checks passed
@NamgungJongMin NamgungJongMin deleted the 19-feat-create-login-page branch January 8, 2024 07:25
HOOOO98 pushed a commit that referenced this pull request Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Feat : create login page
4 participants