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

640 방 나가기 api #677

Merged
merged 11 commits into from
Mar 5, 2024
Merged

640 방 나가기 api #677

merged 11 commits into from
Mar 5, 2024

Conversation

ghyen
Copy link
Contributor

@ghyen ghyen commented Mar 4, 2024

📌 개요

  • 방 나가는 api

💡Issue 번호

@ghyen ghyen requested review from AreSain, JaBeast and yes-ee March 4, 2024 13:03
@ghyen ghyen assigned ghyen and unassigned yes-ee, JaBeast and AreSain Mar 4, 2024
@ghyen ghyen added the enhancement New feature or request label Mar 4, 2024
Copy link
Contributor

@AreSain AreSain left a comment

Choose a reason for hiding this comment

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

API 작성하시느라 고생 많으셨습니다.
수정하실 부분 마저 수정하시고 commit 부탁드려요.
익셉션 처리는 그냥 넘기시면 한번에 바꾸겠습니다

@AreSain AreSain force-pushed the 640-방-나가기-API branch from cad21d9 to 4220bf5 Compare March 4, 2024 21:21
@codecov-commenter
Copy link

codecov-commenter commented Mar 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.22%. Comparing base (e9225ee) to head (37970df).
Report is 3 commits behind head on dev.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #677      +/-   ##
============================================
+ Coverage     74.16%   74.22%   +0.05%     
+ Complexity      737      733       -4     
============================================
  Files           166      164       -2     
  Lines          3178     3162      -16     
  Branches        340      337       -3     
============================================
- Hits           2357     2347      -10     
+ Misses          677      674       -3     
+ Partials        144      141       -3     
Flag Coverage Δ
integrationTest 68.31% <ø> (+0.02%) ⬆️
unitTest 29.34% <ø> (+0.14%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

JaBeast
JaBeast previously approved these changes Mar 4, 2024
@JaBeast JaBeast self-requested a review March 4, 2024 22:35
JaBeast
JaBeast previously approved these changes Mar 5, 2024
Copy link
Contributor

@JaBeast JaBeast left a comment

Choose a reason for hiding this comment

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

.

AreSain
AreSain previously approved these changes Mar 5, 2024
Copy link
Contributor

@AreSain AreSain left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다.

@ghyen ghyen dismissed stale reviews from AreSain and JaBeast via 37970df March 5, 2024 07:56
@ghyen ghyen requested a review from JaBeast March 5, 2024 08:04
- 참여 인원 0명 됐을 때 저장 안 되는 에러 수정
- 유저의 참여 방 여러 개 일 때 다른 방에서 나가는 에러 수정
Copy link
Contributor

@yes-ee yes-ee left a comment

Choose a reason for hiding this comment

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

커멘트 내용 같이 보면서 수정했습니다

@yes-ee yes-ee requested review from yes-ee and AreSain March 5, 2024 11:34
@@ -1,6 +1,7 @@
package gg.data.party;

import java.time.LocalDateTime;
import static gg.utils.exception.BusinessChecker.*;
import static gg.utils.exception.ErrorCode.*;
Copy link
Contributor

Choose a reason for hiding this comment

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

static 처음보네요 음 뭐 없으면 안된다니 참고하겠습니다

Copy link
Contributor

@JaBeast JaBeast left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@ghyen ghyen merged commit d57611c into dev Mar 5, 2024
2 checks passed
@ghyen ghyen deleted the 640-방-나가기-API branch March 5, 2024 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ [Feature] 방 나가기 API
5 participants