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

✨ [STMT-90] 내 정보 수정 API 구현 (#72) #73

Merged
merged 1 commit into from
Mar 4, 2024
Merged

✨ [STMT-90] 내 정보 수정 API 구현 (#72) #73

merged 1 commit into from
Mar 4, 2024

Conversation

zxcv9203
Copy link
Collaborator

@zxcv9203 zxcv9203 commented Mar 4, 2024

#72 참고 부탁드립니다.

* 🔒 [STMT-90] 최초 로그인시 호출할 수 있는 API 제한

* ✨ [STMT-90] 프로필 부분 수정을 위한 값이 입력되지 않았거나 유효한 값인지 판별하는 어노테이션 구현

* ✨ [STMT-90] 유효한 이미지 타입인지 확인하는 기능 구현

* ♻️ [STMT-90] 수정 API를 위해 전달하는 값이 NULL인 경우 기존 값을 그대로 적용하도록 수정

* ♻️ [STMT-90] 요청 DTO 패키지 이동 및 검증 조건 추가

* ♻️ [STMT-90] 회원 프로필 관련 기능을 담당하는 MemberProfileUseCase 생성 및 관련 기능 이관

* 🐛 [STMT-90] 검증 조건 수정으로 인한 테스트 검증 조건 수정

* ✨ [STMT-90] 내 정보 수정 API 구현

* ✅ [STMT-90] 내 정보 수정 API 테스트 코드 추가

* ♻️ [STMT-90] 하나의 메서드가 너무 거대해지지 않도록 회원가입과 내 프로필 수정 메서드를 분리

* ✅ [STMT-90] 프로필 수정 기능 유닛 테스트 추가
@zxcv9203 zxcv9203 requested a review from 05AM as a code owner March 4, 2024 12:41
Copy link
Collaborator

@05AM 05AM left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@zxcv9203 zxcv9203 merged commit ff4a379 into main Mar 4, 2024
1 check passed
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