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

DRAW-217 플레이 중인 방에 모두 나가면 해당 방 삭제 #56

Merged
merged 6 commits into from
Aug 13, 2024

Conversation

comforest
Copy link
Contributor

Related Jira ✔

Description ✔

PR Rule ✔

P1: 꼭 반영해주세요 (Request changes)
P2: 적극적으로 고려해주세요 (Request changes)
P3: 웬만하면 반영해 주세요 (Comment)
P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
P5: 그냥 사소한 의견입니다 (Approve)

@SunwoongH SunwoongH changed the title DRAW-217 fix: 플레이 중인 방에 모두 나가면 해당 방 삭제 DRAW-217 플레이 중인 방에 모두 나가면 해당 방 삭제 Aug 12, 2024
@@ -65,7 +70,6 @@ internal class MafiaGameRoomService(

if (gameInfo.room.players.isEmpty()) {
mafiaGameRepository.removeGameInfo(gameInfo)
return
Copy link
Member

Choose a reason for hiding this comment

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

P5
return 문 제거하신 이유가 따로 있으신걸까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

음 먼가 로직상 있어야 할 것 같은데 왜 없어졌을까요 hmm...
다시 추가할게요 ㅋㅋㅋㅋ

Base automatically changed from feature/DRAW-216 to develop August 13, 2024 11:08
@comforest comforest merged commit 5c18d90 into develop Aug 13, 2024
1 check passed
@comforest comforest deleted the feature/DRAW-217 branch August 13, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants