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/firebase authentication #14

Merged
merged 3 commits into from
Mar 18, 2024
Merged

Feat/firebase authentication #14

merged 3 commits into from
Mar 18, 2024

Conversation

SayisMe
Copy link
Member

@SayisMe SayisMe commented Mar 18, 2024

개요

  • Kakao 로그인을 추가하였습니다.

작업 분류

  • 버그 수정
  • 신규 기능
  • 프로젝트 구조변경

작업 상세 내용

1. Auth 폴더

Auth 폴더 내에서 로그인 관련 코드를 작성할 수 있도록 구분하였습니다.

2. 로그인 컴포넌트

찬섭님의 캘린더 레이아웃을 참고하여 버튼 클릭에 따라 Query Params를 변경하여 login과 signup 컴포넌트를 표시할 수 있도록 하였습니다.
아직 임시방편이기 때문에 추후 변경될 필요가 있습니다.

3. Auth type 구분

login, signup, logged 세 가지로 type을 구분하였습니다.

개선 필요

로그인 완료 이후, 로그인이 되었음을 인지할 수 있도록 유저의 정보를 표시하는 등의 작업이 필요합니다.

Copy link
Contributor

@entrolEC entrolEC 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

@vinivin153 vinivin153 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다👍🏼

@SayisMe SayisMe merged commit aee96e8 into main Mar 18, 2024
3 of 4 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.

3 participants