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

[BE] refactor: 상품 랭킹 조회 시 Response에 카테고리 타입 추가 #421

Merged
merged 3 commits into from
Aug 14, 2023

Conversation

Go-Jaecheol
Copy link
Collaborator

Issue

✨ 구현한 기능

상품 랭킹 조회 시 Response에 카테고리 타입 추가

  • RankingProductDtocategoryType 추가
  • CategoryType enum에 CategoryType -> String(LowerCase) 로 변환해주는 메서드 추가

📢 논의하고 싶은 내용

  • dto 변환해줄 때 categoryType을 product.getCategory().getType().name().toLowerCase() 형태로 받으면 너무 길어지고 복잡한 것 같아서, CategoryType enum에 소문자 문자열로 변환해주는 static 메서드를 만들었는데 어떻게 생각하시나요?
    • (CategoryType에서 변환해주기 vs Category에서 변환해주기)

🎸 기타

x

⏰ 일정

  • 추정 시간 : 15min
  • 걸린 시간 : 10min

@github-actions
Copy link

github-actions bot commented Aug 12, 2023

Unit Test Results

141 tests   141 ✔️  11s ⏱️
  78 suites      0 💤
  78 files        0

Results for commit 69f763b.

♻️ This comment has been updated with latest results.

Copy link
Member

@70825 70825 left a comment

Choose a reason for hiding this comment

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

저도 우가, 망고 코멘트 달은 부분 수정하면 좋을 것 같아요!
고생하셨습니다~

Copy link
Collaborator

@hanueleee hanueleee left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~~ 굿굿👍

@Go-Jaecheol Go-Jaecheol merged commit ab3c93d into develop Aug 14, 2023
3 checks passed
@Go-Jaecheol Go-Jaecheol deleted the feat/issue-383 branch August 14, 2023 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] 상품 랭킹 조회 시 Response에 카테고리 타입 추가
4 participants