feat(user): refresh profile onboarding#55
Open
popeye0618 wants to merge 1 commit into
Open
Conversation
There was a problem hiding this comment.
Code Review
이번 풀 리퀘스트는 회원 탈퇴 시 프로필 데이터 마스킹 로직을 강화하고, 탈퇴 이벤트를 트랜잭션 커밋 이후에 발행하도록 개선하여 시스템의 안정성을 높였습니다. 또한 실명 조회 API 추가, 더미 데이터 생성 기능 고도화, 기본 프로필 이미지 로직 단순화 등의 변경사항이 포함되었습니다. 리뷰 피드백으로는 API 응답 내 contactFrequency 반환 형식이 Enum 명칭으로 변경됨에 따른 프론트엔드와의 협의 필요성과, 런타임 NPE 방지를 위한 방어적인 null 체크 추가 제안이 있었습니다.
| .university(profile.getUniversity()) | ||
| .major(profile.getMajor()) | ||
| .contactFrequency(profile.getContactFrequency().getCode()) | ||
| .contactFrequency(profile.getContactFrequency().name()) |
There was a problem hiding this comment.
API 응답에서 contactFrequency를 기존의 한글 코드(getCode()) 대신 Enum 명칭(name())으로 반환하도록 변경되었습니다. 이는 API 계약(Contract)이 변경되는 사항이므로 프론트엔드 팀과의 공유가 필요합니다. 또한, 도메인 모델 상 필수값이더라도 런타임 NPE 방지를 위해 방어적인 null 체크를 추가하는 것을 권장합니다.
Suggested change
| .contactFrequency(profile.getContactFrequency().name()) | |
| .contactFrequency(profile.getContactFrequency() != null ? profile.getContactFrequency().name() : null) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
개요
회원 프로필/온보딩 흐름을 보강하고 기본 프로필 이미지 리소스와 더미 유저 시딩 정책을 정리합니다.
@codex
변경 사항
테스트
./gradlew :common-module:test --tests com.comatching.common.domain.enums.ContactFrequencyTest./gradlew :user-service:test --tests com.comatching.user.domain.auth.service.SignupServiceImplTest --tests com.comatching.user.domain.member.dto.ProfileUpdateRequestTest --tests com.comatching.user.domain.member.entity.ProfileTest --tests com.comatching.user.domain.member.service.MemberServiceImplTest --tests com.comatching.user.domain.member.service.ProfileServiceImplTest체크리스트
스크린샷 / 참고 자료
feat/product-catalog-bundle(feat(product): add bundle flag to product catalog #50)