Feature/refactor profile image domain split #106
Merged
Conversation
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
- userdataProto → userDataProto로 변경하여 네이밍 오타 수정
- UserRepository 인터페이스와 구현체에서 setNickname, setProfileImage 메서드를 제거 - 닉네임과 프로필 이미지를 한 번에 설정하는 setUserProfile 메서드로 통합 - 사용되지 않는 setEmail 메서드 제거
- 기존 DefaultAuthRemoteDataSource 클래스를 KtorAuthRemoteDataSource로 변경하여 Ktor 기반 구현체임을 명확히 표현
- UserProfileImage → PendingProfileImage: 회원가입/업로드 전 임시 프로필 이미지 모델로 분리 - MemberProfileImage → ProfileImage: 시스템 전반에서 사용하는 도메인 프로필 이미지로 통일 - 관련 파일/클래스/매퍼/확장 함수/사용처 일괄 수정 - UserDataStore에서 프로필 이미지 저장 방식을 파일 경로 → URL로 변경(oneof 스키마 반영)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
PULL REQUEST
Description
프로필 이미지 도메인/모델 정리
MemberProfileImage→ProfileImage(도메인 전역에서 사용하는 사용자 프로필)ProfileImage.Avatar(type, backgroundColor)ProfileImage.Photo(url)← 서버가 내려주는 URL만 보관UserProfileImage→PendingProfileImage(가입/업로드 전 임시 모델)PendingProfileImage.Avatar(...)PendingProfileImage.Photo(file)← 갤러리/카메라로 고른 로컬 파일AuthRemoteDataSource 리네임
DefaultAuthRemoteDataSource→KtorAuthRemoteDataSourceDataStore/Proto 영속 방식 전환
UserDataStore에서 프로필 이미지 파일 경로 대신 URL 저장oneof profile_image { AvatarProto avatar; string file_path; }구조 유지(서로 배타)기타
UserDataStore필드명 오타 수정userdataProto→userDataProto