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: 로컬용 로그인 버튼 추가 #254

Merged
merged 8 commits into from
Sep 10, 2024

Conversation

ParkSohyunee
Copy link
Contributor

개요

  • 프론트엔드 개발할 때 사용하는 로컬용 API를 사용하여 로그인 기능을 구현했습니다.

작업 사항

  • 기존 로컬 로그인 방법

    • 우리 서비스는 소셜로그인만 하고 있으므로 아이디, 패스워드 입력하는 로그인 페이지가 따로 없음
    • 개발할 때 필요한 로컬용 로그인을 하기위해 백엔드에서 개발한 로컬용 로그인 API 적용 후, 토큰 및 사용자 아이디를 쿠키 및 로컬스토리지에 개발자가 직접 넣는 방식
  • 개선한 부분

    • 기존 사용하던 로그인 모달에 개발용 로그인 버튼을 추가 (*개발환경일때만 보이는 버튼입니다.)
    • 로그인 후 토큰 저장 방식 및 관리 방식은 기존 로직을 유지 (*accessToken(AT) 만료 후, refreshToken으로 AT를 새로 받아옵니다.)
    • 로컬용 로그인 후, 로그아웃 시 바로 토큰 및 사용자 정보 제거 로직 추가 (*production 로그아웃 시에는 카카오 API 로그아웃 과정이 추가되므로 개발용일때와 분리)
    • ver1.0때 만들어둔 임시 페이지 ('/login')는 제거하였습니다.

참고 사항

  • 로그인 모달을 누르면 다음과 같이 [개발용 로그인] 버튼을 클릭하여 로그인 할 수 있습니다.
  • 백엔드에서 전달 받은 각자의 아이디 및 공통 비밀번호는 .env.local 환경변수로 저장하고 사용합니다.
  • 마이페이지(설정) - 로그아웃 시, 로그아웃 할 수 있습니다.

스크린샷

image

리뷰어에게

  • 확인 부탁드립니다. 파이팅!! ✨

ParkSohyunee and others added 8 commits June 24, 2024 23:00
* feat: layout 컴포넌트의 metadata 수정 - 리스티웨이브 한글 타이틀 추가 및 description 수정

* Feat: 마이리스트, 콜라보리스트 페이지에 Dynamic 메타데이터 설정

* Feat: 리스트 상세 페이지에 Dynamic 메타데이터 설정

* Refactor: 메타데이터 별도 상수로 분리

* Feat: sitemap 설정 및 도메인 url 별도 상수로 분리

* Feat: 트렌딩리스트에 해당하는 리스트의 상세페이지가 sitemap에 추가되도록 설정

* Feat: robots.txt 설정

* Fix: sitemap.ts 트랜딩리스트 호출 방법을 fetch API로 수정

* Fix: sitemap.ts 트랜딩리스트 호출 로직 제거
* feat: 라벨 및 카테고리 클릭시 검색페이지로 이동

* fix: 리스트 상세조회, 카테고리 전체조회 API 수정에 따라 타입 변경
Copy link

vercel bot commented Sep 5, 2024

@ParkSohyunee is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@seoyoung-min seoyoung-min left a comment

Choose a reason for hiding this comment

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

LGGGGGGTM

const res = await axiosInstance.post<UserOnLoginType>('/login/local', {
account: process.env.NEXT_PUBLIC_LOCAL_LOGIN_ID,
password: process.env.NEXT_PUBLIC_LOCAL_LOGIN_PASSWARD,
});
Copy link
Contributor

Choose a reason for hiding this comment

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

소현님 동호님 덕분에 너무 편할 것 같아요!!🥹
먼저 버튼 만들어주셔서 감사합니다!!

@ParkSohyunee ParkSohyunee merged commit bbc504c into 8-Sprinters:dev Sep 10, 2024
1 of 2 checks passed
@ParkSohyunee ParkSohyunee deleted the feature/login-local branch September 10, 2024 01:22
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.

3 participants