Skip to content

#22 비밀번호 재설정#48

Merged
kingkang85 merged 2 commits intodevelopfrom
47-22-비밀번호-재설정
Dec 17, 2024

Hidden character warning

The head ref may contain hidden characters: "47-22-\ube44\ubc00\ubc88\ud638-\uc7ac\uc124\uc815"
Merged

#22 비밀번호 재설정#48
kingkang85 merged 2 commits intodevelopfrom
47-22-비밀번호-재설정

Conversation

@kingkang85
Copy link
Copy Markdown
Contributor

📋 요약

  • 비밀번호 재발급 및 변경 기능

🛠 변경 사항

  • 이번 Pull Request에서 작업한 주요 변경 사항은 다음과 같습니다:
    • 새로운 기능 추가 또는 버그 수정
    • 코드 리팩토링 또는 최적화
    • 문서 업데이트

🔗 관련 이슈

📸 스크린샷 또는 GIF (해당되는 경우)

  • 비밀번호 재발급
    image

  • 메일 전송 완료
    image

  • 비밀번호 변경
    image

✅ 체크리스트

  • 코드가 정상적으로 동작하는지 테스트했습니다.
  • 관련 문서를 작성하거나 업데이트했습니다. (해당되는 경우)
  • 변경 사항을 반영한 테스트 코드를 추가했습니다.
  • 모든 테스트가 성공적으로 통과했습니다.

🛡 테스트 방법

.

📚 추가 참고 사항

임시번호 발급 후 비밀번호 변경 기능이 필요할 것 같아 추가하였습니다.

- 이메일 전송을 위한 의존성 추가
- EmailService에 이메일 전송을 위한 로직 구현
- UserService의 resetPassword 메서드에서 임시 비밀번호 생성

Related to: #47
- PasswordUpdateRequest에서 기존 비밀번호와 새 비밀번호를 받음
- UserService의 updatePassword 메서드에서 비밀번호 변경
- 기존 비밀번호 검증 후 새 비밀번호 저장

Resolves: #47
@uni-zun
Copy link
Copy Markdown
Contributor

uni-zun commented Dec 17, 2024

임시 비밀번호 생성후 이메일 전송하는 로직이 새롭네요 !

Copy link
Copy Markdown
Contributor

@LeeGukgeon LeeGukgeon 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
Copy Markdown
Contributor

@KIM9909 KIM9909 left a comment

Choose a reason for hiding this comment

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

비밀번호 재발급 정말 대단합니다 !!

@yunjaeeun
Copy link
Copy Markdown
Contributor

smtp이용해서 구현한거 좋네요

@kingkang85 kingkang85 merged commit 4bc65de into develop Dec 17, 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