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/135] 매칭 관련 API 리팩토링 및 수정 #137

Merged
merged 10 commits into from
Aug 9, 2024
Merged

[Fix/135] 매칭 관련 API 리팩토링 및 수정 #137

merged 10 commits into from
Aug 9, 2024

Conversation

rimi3226
Copy link
Contributor

@rimi3226 rimi3226 commented Aug 9, 2024 •
edited
Loading

🚀 개요

매칭 관련 API 리팩토링 및 수정하기

🔍 변경사항

  • Member, MatchingRecord domain 폴더 구조 변경
  • Member Tier, MatchingRecord MatchingStatus, MatchingType ENUM 변경

⏳ 작업 내용

  • request camel 맞추기
  • memberRepository save 추가
  • 매칭 상태, 티어, 매칭 모드 ENUM 변경
  • 매칭 게임 스타일 하나도 선택 안했을 경우 예외처리
  • 매너레벨 DB 추가
  • cicd docker 용량 부족 문제 해결

📝 논의사항

@rimi3226 rimi3226 requested review from Eunjin3395 and hzee97 August 9, 2024 11:40
@rimi3226 rimi3226 self-assigned this Aug 9, 2024
@rimi3226 rimi3226 merged commit b60159a into develop Aug 9, 2024
@rimi3226 rimi3226 deleted the fix/135 branch August 9, 2024 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Eunjin3395 Eunjin3395 Eunjin3395 approved these changes

@hzee97 hzee97 hzee97 approved these changes

Assignees

@rimi3226 rimi3226

Labels
🐛 bug 👷 ci ♻️ refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rimi3226 @Eunjin3395 @hzee97