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 : 같이하기(party) 관련 API 구현 #51

Merged
merged 4 commits into from
Sep 22, 2023
Merged

feat : 같이하기(party) 관련 API 구현 #51

merged 4 commits into from
Sep 22, 2023

Conversation

phjppo0918
Copy link
Member

@phjppo0918 phjppo0918 commented Sep 21, 2023

개요

같이하기 기능(Party mode에 대한) 기능을 구현하였습니다
3가지 기능을 구현하였습니다.

  1. Party 생성
  • Party를 생성하고, 생성자가 방장이 됩니다.
  • 참여 코드는 100,000 ~ 999,999 중 무작위 값으로 설정합니다.
  • 중복을 대비하여 참여 코드와, Party Id는 별도로 설정했습니다.
  1. Party 참여
  • 기존 Waiting, Matching과 달리, event를 구독하면 바로 sink를 생성한 후 구독하도록 설정하였습니다.
  • join시 party에 명단을 추가하고, sink event stream을 전달합니다.
  • 다른 참여자들에게도 party에 대한 event를 multicast 합니다.
  1. Party 시작
  • party 시작 시에 battle service에 create 요청을 보낸 후 id를 받아와 party에 저장합니다.
  • Complete로 상태를 변경한 후, party 참여자들에게 이벤트를 전송하고, sink를 종료시킵니다.

향후 계획

  1. PartyEntity에 대한 값 검증
  • 현재 null check 및 별도 state에 대한 예외 설정을 해당 PR에 반영을 안했습니다. 다음 PR에 추가할 계획입니다.
  1. 방장만 게임 시작 할 수 있도록 설정
  2. quit 기능 구현

@phjppo0918 phjppo0918 self-assigned this Sep 21, 2023
@phjppo0918 phjppo0918 added the ✨ 기능 기능을 구현한다. label Sep 21, 2023
@github-actions
Copy link

github-actions bot commented Sep 21, 2023

Unit Test Results

75 tests   75 ✔️  5s ⏱️
29 suites    0 💤
29 files      0

Results for commit e78fc6b.

♻️ This comment has been updated with latest results.

@sonarcloud
Copy link

sonarcloud bot commented Sep 21, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@seong-wooo seong-wooo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

@seong-wooo seong-wooo merged commit f5a37ef into develop Sep 22, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ 기능 기능을 구현한다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants