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

자체 회원가입을 할 수 있다. #73

Merged
merged 30 commits into from
Aug 1, 2024
Merged

자체 회원가입을 할 수 있다. #73

merged 30 commits into from
Aug 1, 2024

Conversation

ieun32
Copy link
Contributor

@ieun32 ieun32 commented Aug 1, 2024

PR 유형 선택

  • ✨ 기능 추가
  • 🐛 버그 수정
  • 🖼️ UI 변경
  • 🫧 코드 리팩토링
  • ✅ 테스트 코드 추가
  • 📦 기타 (문서, CI/CD, 되돌리기, 포맷팅 …)

작업 내용

  • 자체 회원가입 기능 구현
  • 아이디, 비밀번호, 이메일, 닉네임, 생년월일 입력 화면 및 유효성 검사 로직 구현
  • 이메일 인증 기능 구현: emailjs 사용, 6자리 인증 번호 이메일 전송 후 DB 저장, 입력 값과 DB 저장된 값 비교
  • 비밀번호 단방향 해싱: 보안을 위해 비밀번호를 단방향 해싱, salt 값 사용
  • 소셜 회원가입도 닉네임 입력부터 자연스럽게 진행됨을 확인
  • DB에 저장된 인증 정보는 3분이 지났는 지 확인하고 삭제하는 스케줄러 등록 (1분마다 실행됨)

관련 문서

8월 1일 스크럼

리뷰 요구사항 (선택)

  • 컨벤션에 맞지 않는 코드나 리팩토링이 필요한 코드가 있다면 말씀해주세요!
  • 헤헤 화이팅

구현해야 하는 사항 (다음 PR에 반영될 예정)

  • 자체 회원가입 자잘한 UX 추가
  1. 엔터키로 다음 화면 넘어갈 수 있도록 지원
  2. auto focus
  3. 인증 번호 입력 타이머 넣기 (3분 제한)
  • 자체 로그인
  • 디자인

ieun32 added 30 commits July 30, 2024 11:54
@ieun32 ieun32 added 🖼️ Design UI 변경 📦 Chore 패키지 매니저 수정, 그 외 기타 수정 ✨ Feature 기능 추가 labels Aug 1, 2024
@ieun32 ieun32 requested a review from kyr4601 August 1, 2024 01:20
@ieun32 ieun32 self-assigned this Aug 1, 2024
Copy link
Contributor

@kyr4601 kyr4601 left a comment

Choose a reason for hiding this comment

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

고생하셨숩니다아~~~

@kyr4601 kyr4601 merged commit 5a29a68 into develop Aug 1, 2024
2 checks passed
@ieun32 ieun32 deleted the feature/signup branch August 2, 2024 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 Chore 패키지 매니저 수정, 그 외 기타 수정 🖼️ Design UI 변경 ✨ Feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants