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.

Sign up for GitHub

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

Jump to bottom

[feat #103] 크레딧내역 조회 프로필 번호, 시간 추가 #106

Merged
merged 4 commits into from
Sep 14, 2024
Merged

[feat #103] 크레딧내역 조회 프로필 번호, 시간 추가 #106

merged 4 commits into from
Sep 14, 2024
+32 −9

Conversation

dudxo
Copy link
Collaborator

@dudxo dudxo commented Sep 13, 2024

관련 이슈

📑 작업 상세 내용

  • 크레딧 내역 조회 응답 DTO 프로필 번호, 시간 필드 추가
  • 크레딧 내역 조회 QueryDSL 입/출금에 따른 프로필 번호 결정 로직 추가

💫 작업 요약

  • 크레딧 내역 조회 프로필 번호, 시간 관련 로직 추가

🔍 중점적으로 리뷰 할 부분

  • GPT의 도움을 받아 QueryDSL의 CaseBuilder를 사용해봤는데 if문 등 조건식을 이용해서 쓰는 것보다 훨씬 보기 좋네요!

@dudxo dudxo added ✨ feat 기능 추가 ♻️ refactor 코드 기능 유지하되 코드 변경 labels Sep 13, 2024
@dudxo dudxo requested a review from hyun2371 September 13, 2024 07:08
@dudxo dudxo self-assigned this Sep 13, 2024
@dudxo dudxo linked an issue Sep 13, 2024 that may be closed by this pull request
♻️ 크레딧 내역 조회 API - 사용자 프로필 번호 #103
Closed
1 task
@github-actions GitHub Actions
Copy link

Code Coverage

Overall Project 84.84% 🍏
Files changed 100% 🍏

File Coverage
MemberCustomImpl.java 98.83% 🍏

@github-actions GitHub Actions
Copy link

Test Results

 23 files   23 suites   14s ⏱️
101 tests 101 ✅ 0 💤 0 ❌
102 runs  102 ✅ 0 💤 0 ❌

Results for commit dc80545.

hyun2371
hyun2371 approved these changes Sep 14, 2024
View reviewed changes
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.

casebuilder를 사용하니까 가독성이 좋은 것 같아요! 고생 많으셨습니다~

@dudxo dudxo merged commit 3b50802 into dev Sep 14, 2024
3 checks passed
@dudxo dudxo deleted the feat/#103/CreditHistory-ProfileNum branch September 14, 2024 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hyun2371 hyun2371

Assignees

@dudxo dudxo

Labels
✨ feat 기능 추가 ♻️ refactor 코드 기능 유지하되 코드 변경
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

♻️ 크레딧 내역 조회 API - 사용자 프로필 번호
2 participants
@dudxo @hyun2371