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

로그인이 안되있을때 비밀번호 변경기능 추가 #45

Merged
merged 3 commits into from
Dec 15, 2022
Merged

로그인이 안되있을때 비밀번호 변경기능 추가 #45

merged 3 commits into from
Dec 15, 2022

Conversation

dolong2
Copy link
Member

@dolong2 dolong2 commented Dec 15, 2022

💡 개요

  • 사용자가 로그인할때 비밀번호 변경 기능이 필요함

📃 작업내용

  • 이메일 인증이 되어있을때 이메일과 새 비밀번호로 비밀번호 수정

🎸 기타

#44

@dolong2 dolong2 added 1️⃣Priority: High 우선순위 상 ✨ Feature 기능 개발 labels Dec 15, 2022
@dolong2 dolong2 self-assigned this Dec 15, 2022
@dolong2 dolong2 linked an issue Dec 15, 2022 that may be closed by this pull request
비밀번호를 잊었을때 비밀번호 수정 #44
Closed
baekteun
baekteun reviewed Dec 15, 2022
View reviewed changes
src/main/kotlin/com/msg/gauth/domain/auth/presentation/dto/request/PasswordInitReqDto.kt Outdated
Comment on lines 3 to 5
data class PasswordInitReqDto(
val email: String,
val newPassword: String,
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
Member Author

Choose a reason for hiding this comment

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

반영했음

baekteun
baekteun approved these changes Dec 15, 2022
View reviewed changes
Copy link
Member

@baekteun baekteun left a comment

Choose a reason for hiding this comment

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

코드리뷰

@dolong2 dolong2 merged commit 92734be into main Dec 15, 2022
@dolong2 dolong2 deleted the feature/reset_password branch December 15, 2022 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baekteun baekteun baekteun approved these changes

Assignees

@dolong2 dolong2

Labels
1️⃣Priority: High 우선순위 상 ✨ Feature 기능 개발
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

비밀번호를 잊었을때 비밀번호 수정
2 participants
@dolong2 @baekteun