-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/sch/login page2 #94
Merged
Merged
Conversation
This file contains 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
app.tsx에서 쿠키를 활용함
typescript조건을 맞춘 껍데기
서버로부터 쿠키에 저장된 토큰을 검증받고, 사용자 정보를 반환받아, 사용자 정보를 리듀스의 state타입에 맞는 객체로 만들어 준다
LoginUser의 객체를 바꿔줌
로그인 버튼을 클릭한 후, 로그인 성공시 상태변화를 일으켜 메인부터 다시 로드 되도록 할 필요가 있음 로그인 성공시 반환되는 값은 oid값 뿐이므로,login 액션은 상태에서 isLogin값만 변화시킴
로그인이 되어있는데 이름값이 없는 경우만 유저 상태정보를 패칭하도록 바꿈 한번만 패칭
util의 response를 반환해서 일관성을 맞춤 변경된 반환값에 맞춰 로그인 기능 수정 일관성!!
커밋 실수 ㅠㅠ
성공시 로그인 페이지로 실패시 회원가입 페이지 그대로 있음
jwt 토큰을 검증하고 디코드 하는 미들웨어를 만들고 다른 api들이 이 미들웨어를 활용하도록 구조를 개선함 토큰을 검증하고 디코드하는 과정을 공유하기 쉬움
서버에서 oauth 버튼을 불러오는 API 삭제 oauth 적용
useEffect 안쪽에서 비동기적으로 일이 질행되도록 변경함 메인 화면 로딩 속도가 개선됨
없어진 함수를 import 하는 문제
반환받은 객체가 빈 객체일때 빈객체를 false값으로 간주하고 코딩해서 생긴 에러
로그인 되면 헤더가 변경됨
app.tsx 를 간결하게
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.
로그인 페이지 최종 +
사용자 정보 모든 컴포넌트에서 받을수 있어요
(oauth 사소한 에러.. 수정할 예정임)
사용자 정보 받는법 위키