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

#304 feat: 로그인 로직 변경 #304

Open
wants to merge 319 commits into
base: develop
Choose a base branch
from

Conversation

One-HyeWon
Copy link
Member

@One-HyeWon One-HyeWon commented Dec 18, 2024

1. 무슨 이유로 코드를 변경했나요?

기존 자체 로그인에서 카카오 로그인으로 로직을 변경하였습니다.


2. 어떤 위험이나 장애를 발견했나요?

신규유저와 기존유저의 구분이 필요합니다.
신규 회원가입 클릭시 카카오 로그인으로 연동되는 것은 동일하나, 이후 리다이렉트 되는 페이지가 구분 되어야 합니다.


3. 관련 스크린샷을 첨부해주세요.

image image image image image

4. 완료 사항

  • 랜딩 페이지 로그인 버튼 수정
  • 카카오 로그인 연동
  • 기존 회원과 신규 회원 구분하여 라우팅
  • 토큰 리프레시 로직 연결
  • 신규 회원 가입 로직 연결

5. 추가 사항

close #301


@One-HyeWon One-HyeWon added the Feat 기능 개발 label Dec 18, 2024
@One-HyeWon One-HyeWon self-assigned this Dec 18, 2024
@One-HyeWon One-HyeWon changed the title feat #301 로그인 로직 변경 #304 feat: 로그인 로직 변경 Dec 18, 2024
const navigate = useNavigate();
const CLIENT_ID = import.meta.env.VITE_KAKAO_CLIENT_ID;
const REDIREC_URI = import.meta.env.VITE_KAKAO_REDIRECT_URI;
Copy link
Member

Choose a reason for hiding this comment

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

오타가 있어욤 REDIRECT !!

Copy link
Member Author

Choose a reason for hiding this comment

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

이럴수가! 꼼꼼한 검토 감사합니다 ☺️

Copy link
Member Author

@One-HyeWon One-HyeWon left a comment

Choose a reason for hiding this comment

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

rebase 때문에 변경 파일이 많아졌습니다.... 죄송합니다 ㅜㅡㅜ

Copy link
Member Author

Choose a reason for hiding this comment

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

@yezzan9 @woneeeee @JIN921 @dalzzy
axios 인스턴스를 만들어두었습니다! 인터셉트를 위해 작성 해두었고, 제가 작업하던 브랜치 내 api 폴더 속 파일들은 전부 적용 해 두었습니다. 적용이 되지 않은 파일, 혹은 제가 request/response를 잘못 적용한 파일은 확인 후 수정해주시면 감사하겠습니다!

Copy link
Member

@yezzan9 yezzan9 left a comment

Choose a reason for hiding this comment

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

와웅,,, 엄청 많은 것을 하셨군여 rebase 하면서 추가된 API hook들은 추후에 반영해서 수정해두도록 할게요~! 그 외에 제가 담당했던 API부분엔 문제 없는 것 같아요!

@yezzan9
Copy link
Member

yezzan9 commented Feb 3, 2025

여기 파란색 글자 왼쪽만 여백이 넓은 것 같은데 확인 부탁드려요!
image
image

@One-HyeWon
Copy link
Member Author

여기 파란색 글자 왼쪽만 여백이 넓은 것 같은데 확인 부탁드려요! image image

헉!!!!!!!!! 매의 눈 감사합니다,,,,,,,,,,,,, p 태그 때문인지 띄어쓰기가 안 먹혀서 공백 문자를 넣었더니 기준이 다른가봐요! 다른 방법을 찾아보도록 하겠습니다!

Copy link
Member

@dalzzy dalzzy left a comment

Choose a reason for hiding this comment

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

확인했습니다 !! 수고하셨습니다 ~~

Copy link
Member

Choose a reason for hiding this comment

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

파일명에 오타가 있어욤

flex-direction: column;
width: 370px;
max-width: 370px;
padding-top: 0;
Copy link
Member

Choose a reason for hiding this comment

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

여기 설정된 padding-top: 0 도 제거하는 걸 깜빡하신거겠죠??

Copy link
Collaborator

@JIN921 JIN921 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다🥹

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 로그인 로직 변경
5 participants