Skip to content

Feature: Credential Login 구현 #74

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

Merged
merged 5 commits into from
Oct 2, 2023
Merged

Feature: Credential Login 구현 #74

merged 5 commits into from
Oct 2, 2023

Conversation

soulchicken
Copy link
Member

PR 제목

PR을 한 이유 🎯

  • 백엔드에서 만들어진 로그인 API를 연결했습니다.

이슈 번호 📎

변경사항 🛠

  • 기존 땜빵해놨던 next 자체 로그인 기능에서 백엔드로 연결하도록 설정했습니다.

특이사항 📌

  • 아직 서버측 dev에만 적용되어있는 것이 특징입니다.
  • 아직 소셜로그인은 되지 않았습니다.

테스트 결과 📝

  • �로그인, 회원가입 기능이 로컬에서 잘 작동하는 것을 확인했습니다.

- page에서 api 보내기 전에 비밀번호 유효성 검사를 하지 않고 넥스트 서버 측에서 api 콜 하기 전에 비밀번호 일치 여부를 확인하도록 했습니다
- axios에서 중복된 header application/json을 인스턴스 정의할 때 관리하도록 했습니다.
@soulchicken
Copy link
Member Author

지금 유저 name 부분에 숫자가 되지 않게 서버에서 적용되어 있는데 회의 때 대화할만한 주제입니다.

@soulchicken soulchicken merged commit 50af011 into TeamATM:develop Oct 2, 2023
@soulchicken soulchicken deleted the feature/login-api branch October 2, 2023 16:32
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.

기본 로그인 백엔드 API 소통 연결
1 participant