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

Refactor #75 signup #82

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Refactor #75 signup #82

wants to merge 7 commits into from

Conversation

LimiteDiTempo
Copy link
Contributor

연관된 이슈 번호

#81

개요

회원 가입 전 이메일 인증

작업 사항

  • smtp 이메일 전송 로직 구현, redis에 코드 저장

Copy link
Member

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

@pea06 pea06 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.

이 파일은 뭔가요..?

Copy link
Contributor Author

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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants