Feature(#236): 로그인/회원가입 페이지 작업 #239
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- login 페이지 이름을 UserAuth로 변경하였습니다. - App.tsx 코드를 순서에 맞게 정리하였습니다.
- 로그인/회원가입 페이지 작업을 위한 이미지를 추가하였습니다.
- 로그인 UI 작업을 완료하였습니다.
- 회원가입 UI 작업을 완료하였습니다.
- 각각의 입력 필드에 대한 검증을 진행하고, 문제가 없을 경우 이를 표시하도록 처리하였습니다.
- 로그인과 회원가입 API 연동을 완료하였습니다.
- login 페이지 이름을 UserAuth로 변경하였습니다. - App.tsx 코드를 순서에 맞게 정리하였습니다.
- 로그인/회원가입 페이지 작업을 위한 이미지를 추가하였습니다.
- 로그인 UI 작업을 완료하였습니다.
- 회원가입 UI 작업을 완료하였습니다.
- 각각의 입력 필드에 대한 검증을 진행하고, 문제가 없을 경우 이를 표시하도록 처리하였습니다.
- 로그인과 회원가입 API 연동을 완료하였습니다.
✅ Deploy Preview for boarlog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
- 잘못 변경한 state값을 수정하였습니다.
Jw705
approved these changes
Dec 9, 2023
|
|
||
| const handlePasswordConfirm = (e: React.ChangeEvent<HTMLInputElement>) => { | ||
| setPasswordConfirm(e.target.value); | ||
| setIsPasswordConfirm(e.target.value === password); |
There was a problem hiding this comment.
사소한 내용이긴 하지만, 이 조건문은 "비밀번호"와 "비밀번호 확인" input 태그가 둘다 비어있어도 true를 반환하는 것 같아요! e.target.value에 내용이 있는지 확인하는 내용도 있으면 더 좋을 것 같습니다!
There was a problem hiding this comment.
생각해보니 맞네요..! value에 값이 있는지를 판단해서 띄워주도록 해야할 것 같습니다 ㅠㅠ
Byeonjin
reviewed
Dec 9, 2023
| <h1 className="semibold-64">Boarlog</h1> | ||
| <h2 className="semibold-20 text-grayscale-darkgray">기록으로 남기는 실시간 강의</h2> | ||
| </div> | ||
| {isSignIn ? <></> : <></>} |
There was a problem hiding this comment.
이 부분은 무엇을 위한 건지 궁금하네요! ㅎㅎ
There was a problem hiding this comment.
헉...! SignIn인지 판단하기 위해서 복붙해서 사용할 코드를 그냥 두었네요... ㅠㅠㅠ 제거하겠습니다 :)
….com/LellowMellow/web13_Boarlog into feature/231209-change-signin-up-page
- 비밀번호 확인 시에 해당 빈칸에 값이 들어있는지의 여부를 판단하도록 하였습니다. - 추가적으로 불필요한 코드를 제거하였습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업 개요
작업 사항
고민한 점들(필수 X)
현재 구글 로그인을 대신하여 로그인을 진행하기 때문에 최대한 간단하게 구현하기 위해 로그인과 회원가입을 한 페이지에서 처리할 수 있도록 하였습니다. 우선, 로그인과 회원가입을 한 곳에서 처리하도록 UI 작업을 진행하였습니다.
API 요청에 따른 결과값에 따라 Toast 표시와 더불어 페이지 이동과 같은 추가적인 로직을 반영하여 작업을 완료하였습니다.
스크린샷(필수 X)
로그인/회원가입 전환
2023-12-09.20-28-22.mp4
로그인/회원가입 반응형 처리
2023-12-09.20-28-01.mp4
회원가입 입력값 유효 여부 판단/신규 유저 가입 성공
2023-12-09.20-32-10.mp4
존재하는 유저 가입 시도 처리
2023-12-09.20-31-43.mp4
로그인 입력값 유효 여부 판단
2023-12-09.20-30-58.mp4
존재하지 않는 유저 로그인 시도 처리
2023-12-09.20-31-19.mp4
로그인 성공
2023-12-09.20-29-56.mp4