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.

Sign up for GitHub

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

Jump to bottom

비밀번호 변경 기능 추가 구현 #169

Merged
merged 4 commits into from
Jun 11, 2024
Merged

비밀번호 변경 기능 추가 구현 #169

merged 4 commits into from
Jun 11, 2024

Conversation

JadeKim042386
Copy link
Member

@JadeKim042386 JadeKim042386 commented Jun 11, 2024 •
edited
Loading

🔍️ 이 PR을 통해 해결하려는 문제

  • 사용자가 다른 비밀번호로 변경할 수 있도록 비밀번호 변경 기능을 추가합니다.

✨ 이 PR에서 핵심적으로 변경된 사항

  • 비밀번호 변경 기능 구현

    • 기존 비밀번호와 변경할 비밀번호를 전달받고 입력한 기존 비밀번호와 DB 상 저장되어있는 비밀번호를 확인
    • 비밀번호가 일치하지 않으면 예외(MemberException) 발생
    • 비밀번호 업데이트 로직을 update 쿼리를 사용하여 간단하게 구현

    image

  • 단위 테스트 코드 작성

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분

  • 메일 인증 코드 불일치에 대한 에러코드 네이밍 수정

    • AUTH_CODE_UNMATCHED -> UNMATCHED_AUTH_CODE

  • LoginRequest의 password에도 validation 적용

📌 PR 진행 시 이러한 점들을 참고해 주세요

  • Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
  • Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
  • Review는 특수한 케이스가 아니면 Reviewer로 지정된 시점 기준으로 1일 이내에 진행해 주세요.

Issue Tags

@JadeKim042386
feat: #168 - 비밀번호 기능 구현
de478a4 
- 기존 비밀번호와 변경할 비밀번호를 전달받고 입력한 기존 비밀번호와 DB 상 저장되어있는 비밀번호를 확인
- 비밀번호가 일치하지 않으면 예외 발생
- 비밀번호 업데이트 로직을 update 쿼리를 사용하여 간단하게 구현
@JadeKim042386 JadeKim042386 added the enhancement New feature or request label Jun 11, 2024
@JadeKim042386 JadeKim042386 self-assigned this Jun 11, 2024
@JadeKim042386 JadeKim042386 merged commit 3b3ce07 into develop Jun 11, 2024
@JadeKim042386 JadeKim042386 deleted the feature/#168-change-password branch June 11, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

Assignees

@JadeKim042386 JadeKim042386

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

비밀번호 변경 기능 추가
1 participant
@JadeKim042386