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/181] 매칭 found API 구현 및 우선순위 계산 API 응답값 수정 #186

Merged
merged 4 commits into from
Aug 23, 2024

Conversation

Eunjin3395
Copy link
Member

@Eunjin3395 Eunjin3395 commented Aug 22, 2024

🚀 개요

매칭 found API 구현 및 우선순위 계산 API 응답값 수정

🔍 변경사항

  • 매칭 found API 구현
  • 우선순위 계산 API 응답에 나의 매칭 요청 데이터 포함
  • profileService에 gameStyle string list 반환 메소드 추가

⏳ 작업 내용

  • 매칭 found API 구현
  • 우선순위 계산 API 응답에 나의 매칭 요청 데이터 포함

📝 논의사항

@@ -29,7 +29,16 @@ public class MatchingController {
private final MatchingService matchingService;

@PostMapping("/priority")
@Operation(summary = "우선순위 계산 및 매칭 기록을 저장하는 API 입니다.", description = "API for calculating and recording matching")
@Operation(summary = "우선순위 계산 및 매칭 기록을 저장하는 API 입니다.", description =
"API for calculating and recording matching \n\n"
Copy link
Contributor

Choose a reason for hiding this comment

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

이렇게 request 값 알려주는거 좋은 것 같아욥!

Copy link
Member Author

Choose a reason for hiding this comment

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

감사합니다! 그런데 생각해보니 포지션 3종은 05까지 가능인데 제가 15까지로 써놨더라구요... 다음 pr 하면서 수정하겠습니다..!

Tier tier;
Integer rank;
Integer mannerLevel;
Integer profileImg;
Copy link
Contributor

Choose a reason for hiding this comment

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

저희 프로필 이미지도 따로 저장해야할 이유가 있었나요?

Copy link
Contributor

Choose a reason for hiding this comment

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

생각해보니까 매칭할 때 프로필 이미지를 보여줘야해서 필요할 것 같네요..!!!

@rimi3226
Copy link
Contributor

고생하셨습니다!!

@Eunjin3395 Eunjin3395 merged commit 98f66b0 into develop Aug 23, 2024
1 check passed
@Eunjin3395 Eunjin3395 deleted the feat/181 branch August 23, 2024 12:49
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.

✨ [Feat] 매칭 found API 구현 및 우선순위 계산 API 응답값 수정
2 participants