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

Fix: 나의 뱃지 목록 조회 응답 변경 사항에 맞게 수정 #292

Merged
merged 12 commits into from
Oct 31, 2024
Merged

Fix: 나의 뱃지 목록 조회 응답 변경 사항에 맞게 수정 #292

merged 12 commits into from
Oct 31, 2024

Conversation

hee9841
Copy link
Collaborator

@hee9841 hee9841 commented Oct 30, 2024 •
edited
Loading

🔗 이슈 연결

🚀 구현한 API

  • X

💡 반영할 내용 및 변경 사항 요약

  • DTO 변경사항
    • AchievedBadgesResponse의 AchievedBadge를 공통으로 사용하기 위해 분리합니다.
    • 응답 형식의 변경으로 사용하지 DTO는 삭제합니다.
  • 리파지토리 변경사항
    • 기존에 뱃지 목록을 가져오는 함수를 사용하지 않아 JooqBadgeRepository를 삭제합니다.
    • 뱃지 목록을 가져오는 함수를 BadgeAchievementRepository에 추가합니다.
  • BadgeType 변경 사항
    • 뱃지 타입의 이름을 리턴하기 위해 타입 이름을 나타내는 name를 추가합니다.
    • 리턴 시 보여지는 타입을 순서(개인기록 -> 러닝거리 -> ..순으로)로 리턴하기 위해 'showPriority'을 추가합니다.

🔍 리뷰 요청/참고 사항

  • 수정하다보니 파일이 많이 변경되었어요.
  • 리턴 시 피그마의 뱃지 타입순 으로 리턴하는 게 좋을 것 같아서 배지 타입에 우선순위를 지정해서 리턴할 수 있게 했습니다. 해당 사항은 팀 전체에게 여쭤보고 바꿔질 수 있을 것 같아요.

hee9841 added 10 commits October 30, 2024 19:34
@hee9841
Refactor: AchievedBadgesResponse의 AchievedBadge를 공통으로 사용하기 위해 분리
97874b5
@hee9841
Feat: 사용자가 획득한 뱃지를 뱃지 타입순, 최신 획득 순으로 가져오는 repository 추가
2667486
@hee9841
Fix: 뱃지 목록 조회의 새로운 응답 형식 변경에 따른 수정
2576c12 
- AllBadgesListResponse변경
- 기존의 뱃지 목록을 조회하는 repository 삭제
- 필요 없어진 BadgeWithAchieveStatusAndAchievedAt DTO 삭제
- 기존 테스트 코드 삭제
- BagdeType에 타입한글 이름 추가
@hee9841
Fix: 최신 획득 뱃지는 badgeList에 추가하지 않고 따로 리턴하게 수정
1b01e30
@hee9841
Test: findByMemberIdOrderByBadgeTypeAndAchievedAt 리파지토리 테스트
e21ce7d
@hee9841
Test: 나의 벳지 목록 조회 서비스 테스트 코드 추가
abd6528
@hee9841
Fix: 보여주는 우선순위 값 추가
83c0b88
@hee9841
Fix: 벳지의 타입 별 보여주는 우선 순위로 배지 리스트 반환하게 수정
f40d621
@hee9841
Test: findBadgesList의 잘못된 테스트 코드 수정
9a7b17f
@hee9841
Docs: 나의 뱃지 목록 조회 API 문서 수정
e6c71ae
@hee9841 hee9841 self-assigned this Oct 30, 2024
@hee9841 hee9841 changed the title Fix: 나의 뱃지 목록 조회 응답 변경 사항 수정 Fix: 나의 뱃지 목록 조회 응답 변경 사항에 맞게 수정 Oct 30, 2024
@hee9841 hee9841 requested a review from WonSteps October 30, 2024 14:31
@hee9841 hee9841 requested a review from WonSteps October 31, 2024 02:19
@hee9841 hee9841 merged commit 5626eb0 into main Oct 31, 2024
1 check passed
@hee9841 hee9841 deleted the fix/#286/badges-list branch October 31, 2024 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WonSteps WonSteps WonSteps approved these changes

Assignees

@hee9841 hee9841

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

나의 뱃지 목록 조회 응답 변경 사항에 맞게 수정
2 participants
@hee9841 @WonSteps