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

[BE] feat#74#77 회원가입, 로그인 구현 #226

Merged
merged 3 commits into from
Nov 20, 2024

Conversation

songbuild00
Copy link
Collaborator

➕ 이슈 번호


🔎 작업 내용

  • 로그인 구현 (JWT 발급)
  • 회원가입 구현 (이메일, 비밀번호, 닉네임 입력)
  • JwtAuthGuard 체크 (제대로 작동하는지)

🖼 참고 이미지

회원가입

로그인

로그인 토큰 테스트 (프로필)


✅ Check List

  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

Copy link
Collaborator

@DongHoonYu96 DongHoonYu96 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.
이왕 하시는김에 로그인없이 "체험하기" 같은 기능도 만들면 좋겠네요.

Comment on lines +16 to +20
@UseGuards(LocalAuthGuard)
@Post('login')
async login(@Request() req) {
return this.authService.login(req.user);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

로컬 가드는 역할이 뭔가요?

Copy link
Collaborator

Choose a reason for hiding this comment

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

로그인할때 body로 email, pw 얻어와서 jwt로 바꾸는역할

Copy link
Collaborator

@NewCodes7 NewCodes7 left a comment

Choose a reason for hiding this comment

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

건석님 수고하셨습니다!!
처음하는 작업이라 힘드셨을텐데 잘 하셨군요!

email: signupDto.email,
password: hashedPassword,
nickname: signupDto.nickname,
status: '?'
Copy link
Collaborator

Choose a reason for hiding this comment

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

status를 통해 활동 정지 상태, 활성 상태 등 이러한 걸 나타내고자 테이블에 넣었었습니다! 지금 당장은 필요하지 않아서 크게 신경쓰시지 않아도 될 듯 합니다!

Copy link
Collaborator

@always97 always97 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
Collaborator

@ijun17 ijun17 left a comment

Choose a reason for hiding this comment

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

로그인 굿 고생하셨습니다!

@songbuild00 songbuild00 merged commit 1300288 into boostcampwm-2024:dev-be Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants