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

[feat] : 마이페이지 프로필 조회 #94

Merged
merged 10 commits into from
Mar 15, 2024

Conversation

ParkJuhan94
Copy link
Contributor

@ParkJuhan94 ParkJuhan94 commented Mar 14, 2024

close #93
close #95

📑 작업 상세 내용

  • 마이페이지 기본 정보 조회 API
    • 프로필 이미지, 닉네임, 동, 선호 카테고리의 List<String>, 점수를 응답으로 반환합니다.
  • 리뷰 등록시에, 사용자의 Score 갱신 추가
    • User 엔티티에 비즈니스 메서드를 만들었습니다.
    • 0-200 점 사이로 점수를 구성하기 위하여 음수 OR 200 초과를 검사하여 다시 갱신합니다.
    • 이 점수를 그대로 응답으로 반환하면, 프론트엔드에서 점수 -> 신뢰도로 변환합니다.

💫 작업 요약

  • 마이페이지 기본 정보 조회 API
  • 리뷰 등록시에, 사용자의 Score 갱신 추가

🔍 중점적으로 리뷰 할 부분

  • UserService
  • UserApiControllerTest
  • User
  • ReviewService
  • ReviewApiControllerTest

@ParkJuhan94 ParkJuhan94 added the feat New feature or request label Mar 14, 2024
@ParkJuhan94 ParkJuhan94 requested a review from hyun2371 March 14, 2024 20:57
@ParkJuhan94 ParkJuhan94 self-assigned this Mar 14, 2024
Copy link

github-actions bot commented Mar 14, 2024

Test Results

117 tests  +1   117 ✅ +1   27s ⏱️ -1s
 29 suites ±0     0 💤 ±0 
 29 files   ±0     0 ❌ ±0 

Results for commit 5df3135. ± Comparison against base commit 0f4a163.

♻️ This comment has been updated with latest results.

Copy link
Member

@hyun2371 hyun2371 left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다! 개인적으로 기본 정보가 잘 안 와닿아서 상세정보나 프로필 조회로 고려해봐도 좋을 것 같아요!

@@ -78,4 +79,14 @@ public ResponseEntity<PageResponse<UserReviewResponse>> getUserReviews(
return ResponseEntity.ok(response);
}

@NoAuth
@GetMapping("/api/users/{userId}/bases")
Copy link
Member

Choose a reason for hiding this comment

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

uri에 bases 대신 info나 detail, mypage를 넣으면 더 직관적일 것 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 위의 코멘트랑 맞춰서 profiles 로 해보겠습니다!

import java.util.List;

public record UserBasicInfoResponse(

Copy link
Member

Choose a reason for hiding this comment

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

userId도 반환해주면 좋을 것 같아요!

@ParkJuhan94 ParkJuhan94 changed the title [feat] : 마이페이지 기본 정보 조회 [feat] : 마이페이지 프로필 조회 Mar 15, 2024
@ParkJuhan94 ParkJuhan94 merged commit d2f3e91 into dev Mar 15, 2024
3 checks passed
@ParkJuhan94 ParkJuhan94 deleted the feat/#93/mypage-basic-info branch March 15, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] : 리뷰 등록시에, 사용자의 Score 갱신 추가 [feat] : 마이페이지 프로필 조회
2 participants