Feat/12 check my profile#17
Merged
Merged
Conversation
- ImageController,ImageProcessingService 구현 및 테스트 코드 작성 - 공통 응답 구조 전용 ApiResponse 클래스 구현 - ai api 활용 시 사용할 dto 레코드 구현 Related: #10
Related: #12
- User 클래스에 Builder 패턴 적용 - UserRequest dto에 uuid 필드 타입 변경(Long -> UUID) - UserService에서 유저 검색 및 반환 메서드에서 검색 실패 시 null 반환 대신 예외 던지도록 수정 - 기존 컨트롤러 테스트 코드가 서비스 계층 쪽 테스트에 치우쳐 있는 걸 확인, 테스트 클래스 이름 변경(controller->service) 후 컨트롤러 테스트 코드 작성
Contributor
|
🚨 PR 본문이 비어있습니다! 아래 템플릿을 복사하여 PR 내용을 작성해주세요. 📝 개요
💻 작업 내용
✅ PR 체크리스트
🔗 관련 이슈
스크린샷 (선택)
|
Collaborator
Author
|
현재 로그인 기능이 없으므로 /users/me 대신 uuid를 사용해 PathVariable로 구현하였습니다 |
Jeonghun01
reviewed
Jul 14, 2025
| this.updatedAt = LocalDateTime.now(); // 최초 생성 시 updated_at도 설정 | ||
| if (this.level == null) this.level = 1L; // 기본 레벨 설정 | ||
| if (this.experiencePoints == null) this.experiencePoints = 0; // 기본 경험치 설정 | ||
| if (this.temperatureScore == null) this.temperatureScore = 0; // 기본 온도 점수 설정 |
Member
There was a problem hiding this comment.
여기 기본 경험치랑 기본 온도 점수가 다르게 해놓으신 거 같은데, 하나로 통일 해주세요
별개 경험치가 아니고 하나의 체제로만 운영될 거에요
Collaborator
Author
There was a problem hiding this comment.
experiencePoints 삭제하고 temperaturescore로 통일했슴다
Member
There was a problem hiding this comment.
근데 미안한데 ‘온도’가 아니라 ‘꽃’ 점수이긴해
원래 온도는 가제였고 꽃으로 하기로 했어
FlowerScore로 통일하실까요..?
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
2 participants
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.
📝 개요
💻 작업 내용
✅ PR 체크리스트
🔗 관련 이슈
스크린샷 (선택)