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

회원가입 페이지 400 에러 해결 #32

Merged
merged 2 commits into from
Jun 13, 2024
Merged

Conversation

imsoohyeok
Copy link
Collaborator

@imsoohyeok imsoohyeok commented Jun 12, 2024

  • 400에러 해결
  • 테스트 (예시: test123, test1234, test123456, test0330 + @codeit.com)

400에러 해결해서 이제는 아마 정상적으로 작동할겁니다!
테스트도 많이 해봤는데 제대로 작동 하는 것 같습니다. (6번 정도)

!! mypage 부분의 UI도 조금 수정했는데 아직 완성은 아니니 안보셔도 될 것 같습니다.
(mypage 하다가 갑자기 에러 고칠 생각이 나서 하다 말고 고치러 간... 하하)

++ 로그인, 회원가입 페이지에서도 로그인 상태시 mydashboard로 이동하게는 아직 구현 안했습니다!

ISSUE

기존의 400에러가 뜨는 이유는 각 함수에서 받아오는 Input의 value값이 달라서 폼을 제출할 때 빈 값이 전달되어서 그랬던 것이었습니다.

가장 큰 변화점은 useInputHandlers 함수, useLoginForm 함수, useSignUpForm 함수의
Input의 value값을 통일 시키기 위해서, useCombinedSignUpForm.ts 파일이 추가되었습니다.
그래서 각 컴포넌트 LoginForm, SignUpForm 에서는 useCombinedSignUpForm로 부터 값을 받아옵니다.

테스트 해 보시고 문제 발생하면 말씀해주세요!

@imsoohyeok imsoohyeok requested review from eunji-0623, ssumai-kr, a user and foxholic9 June 12, 2024 12:16
@eunji-0623 eunji-0623 merged commit 74c051e into dev Jun 13, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants