[Feature] 미팅 전체 취소 api 구현 및 미팅 정책 위반 로직 수정 - #78
Open
Sehi55 wants to merge 19 commits into
Open
Conversation
[FEAT] 연애관 채팅 프로필 공개 요청 구현
[FIX] 채팅방 오류 수정
|
Warning Review limit reached
Next review available in: 19 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Walkthrough미팅 전체 취소 요청·투표·만료 처리를 추가했습니다. 승인 시 미팅과 관련 채팅방을 비활성화합니다. 빠른 매칭 입장·퇴장과 미팅 상태 전이를 보강했습니다. 비활성 채팅방의 메시지, 동기화, 구독을 제한했습니다. Changes미팅 취소 및 매칭 채팅 흐름
Estimated code review effort: 4 (Complex) | ~60 minutes Sequence Diagram(s)sequenceDiagram
participant Client
participant MeetingController
participant MeetingCancellationService
participant MeetingCancellationVoteRepository
participant ChatRoomService
participant Meeting
Client->>MeetingController: 취소 요청 또는 투표
MeetingController->>MeetingCancellationService: 취소 처리 위임
MeetingCancellationService->>MeetingCancellationVoteRepository: 투표 저장 및 집계
MeetingCancellationService->>ChatRoomService: 채팅방 및 멤버 비활성화
MeetingCancellationService->>Meeting: 취소 상태 반영
MeetingCancellationService-->>MeetingController: 취소 응답 반환
Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (2)
manabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationServiceTest.java (1)
124-156: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win채팅방/멤버 비활성화 경로에 대한 검증 보강 권장
unanimousApprovalCancelsBothMatchedTeams는meeting1/meeting2의cancelByAgreement()호출만 검증합니다.approveCancellation()이 함께 수행하는chatRoomRepository.findByMatch/findByMeeting조회 결과에 대한deactivateChatRoomAndMembers호출(채팅방/멤버 비활성화)은 이 테스트에서 커버되지 않습니다. 이번 PR의 목표 중 하나가 "매칭 성사 후 채팅방 비활성화"이므로, 관련 verify를 추가해 회귀를 방지하는 것을 권장합니다.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@manabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationServiceTest.java` around lines 124 - 156, 보 unanimousApprovalCancelsBothMatchedTeams 테스트가 meeting 취소뿐 아니라 채팅방과 멤버 비활성화 경로도 검증하도록 보강하세요. approveCancellation()에서 사용하는 chatRoomRepository의 매칭/회의 조회 결과를 설정하고, deactivateChatRoomAndMembers 호출이 두 매칭 팀에 대해 수행되는지 검증하세요.manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingService.java (1)
184-200: 🎯 Functional Correctness | 🔵 Trivial
enterRoomByCode/enterRoomById로직 중복 및saveAndFlush처리 불일치.두 메서드가 취소 대기 검증 → 입장 검증 →
isFastMatchingEntry계산 →addMember→ 채팅방 조인 → 빠른 입장 매칭방 연결 → 응답 생성까지 거의 동일한 로직을 반복하고 있습니다. 공통 private 메서드로 추출하면 유지보수성이 개선되고, 두 진입점이 항상 동일하게 동작함을 보장할 수 있습니다.추가로
enterRoomByCode(Line 188)에만meetingRepository.saveAndFlush(meeting)호출과 "왜 addMember가 반영이 안되지" 주석이 있는데,enterRoomById에는 동일한 flush가 없습니다. 원래 flush가 필요했던 이유(더티 체킹이 반영되지 않는 문제)가 실재한다면enterRoomById에도 동일한 문제가 잠재해 있을 수 있습니다. 두 진입점에서 flush 필요 여부를 명확히 하고 일관되게 처리해 주세요.Also applies to: 215-249
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingService.java` around lines 184 - 200, Extract the shared flow from enterRoomByCode and enterRoomById into one private helper covering cancellation validation, join validation, fast-matching detection, member addition, chat-room joins, and response construction. Ensure both entry points delegate to this helper and use the same meeting persistence behavior, explicitly deciding whether saveAndFlush is required and applying that decision consistently; remove the temporary comment.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@manabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatRoomService.java`:
- Around line 139-155: Update joinMatchingChatRoom to check duplicate membership
using the member status, limiting the existsBy query to
ChatMemberStatus.ACTIVATE so users with DEACTIVATED membership history can
rejoin; preserve the existing active-member exception and save flow, and add a
database uniqueness safeguard if required to prevent concurrent duplicate
inserts.
In
`@manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationService.java`:
- Around line 155-165: Update expirePendingRequests() so each expired request is
processed in an independent transaction, preventing an OptimisticLockException
for one request from rolling back other requests. Move the per-request expire
operation into a transactional boundary that can commit or roll back
independently, while preserving the existing count and expiration timestamp
behavior.
In
`@manabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/MeetingCancellationStatus.java`:
- Around line 3-8: Synchronize MeetingCancellationStatus with the
chk_cancellation_request_status database constraint by adding the missing
WITHDRAWN enum value, unless the schema is intentionally changed to remove it;
ensure the JPA enum values and allowed database statuses remain consistent.
---
Nitpick comments:
In
`@manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingService.java`:
- Around line 184-200: Extract the shared flow from enterRoomByCode and
enterRoomById into one private helper covering cancellation validation, join
validation, fast-matching detection, member addition, chat-room joins, and
response construction. Ensure both entry points delegate to this helper and use
the same meeting persistence behavior, explicitly deciding whether saveAndFlush
is required and applying that decision consistently; remove the temporary
comment.
In
`@manabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationServiceTest.java`:
- Around line 124-156: 보 unanimousApprovalCancelsBothMatchedTeams 테스트가 meeting
취소뿐 아니라 채팅방과 멤버 비활성화 경로도 검증하도록 보강하세요. approveCancellation()에서 사용하는
chatRoomRepository의 매칭/회의 조회 결과를 설정하고, deactivateChatRoomAndMembers 호출이 두 매칭 팀에
대해 수행되는지 검증하세요.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: de0f7f96-38d9-4777-899a-18c3ec06efb0
📒 Files selected for processing (27)
manabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatRoomService.javamanabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/dto/request/MeetingCancellationVoteRequest.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/dto/response/MeetingCancellationResponse.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/dto/response/MeetingCancellationVoteResponse.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/scheduler/MeetingCancellationScheduler.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingMatchingService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingService.javamanabom/src/main/java/mannabom_server/manabom/domain/chat/repository/ChatMemberRepository.javamanabom/src/main/java/mannabom_server/manabom/domain/chat/repository/ChatRoomRepository.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/entity/Meeting.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/entity/MeetingCancellationRequest.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/entity/MeetingCancellationVote.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/CancellationVoteDecision.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/MeetingCancellationStatus.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/MeetingStatus.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingCancellationRequestRepository.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingCancellationVoteRepository.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingMatchRepository.javamanabom/src/main/java/mannabom_server/manabom/infrastructure/security/websocket/StompAuthChannelInterceptor.javamanabom/src/main/java/mannabom_server/manabom/presentation/meeting/controller/MeetingController.javamanabom/src/main/resources/db/migration/V21__add_meeting_cancellation_tables.sqlmanabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sqlmanabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationServiceTest.javamanabom/src/test/java/mannabom_server/manabom/domain/meeting/MeetingCancellationDomainTest.javamanabom/src/test/java/mannabom_server/manabom/domain/meeting/MeetingMemberLeaveStatusTest.java
There was a problem hiding this comment.
🧹 Nitpick comments (2)
manabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sql (1)
4-11: 🩺 Stability & Availability | 🔵 Trivial | ⚡ Quick win테이블 락(Table Lock) 방지를 위한
NOT VALID활용 권장운영 환경에서 제약 조건을 추가할 때 테이블 전체를 스캔하는 동안 쓰기 작업이 차단(Lock)될 수 있습니다. 이를 방지하기 위해
NOT VALID로 제약 조건을 먼저 추가한 뒤VALIDATE CONSTRAINT를 통해 비동기적으로 검증하는 방식을 권장합니다.💡 제안하는 마이그레이션 스크립트 수정안
ALTER TABLE meeting_cancellation_requests ADD CONSTRAINT chk_cancellation_request_status CHECK (status IN ( 'PENDING', 'APPROVED', 'REJECTED', 'EXPIRED' - )); + )) NOT VALID; + +ALTER TABLE meeting_cancellation_requests + VALIDATE CONSTRAINT chk_cancellation_request_status;🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@manabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sql` around lines 4 - 11, Update the chk_cancellation_request_status constraint in the migration to add it with NOT VALID, then separately validate it using VALIDATE CONSTRAINT so constraint creation does not scan and lock the entire table during the initial ALTER TABLE operation.Source: Linters/SAST tools
manabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sql (1)
1-3: 🩺 Stability & Availability | 🔵 Trivial인덱스 생성 시 테이블 락(Lock) 발생 주의
운영 환경에서 데이터가 많은 테이블에 일반적인 방식으로 인덱스를 생성하면, 인덱스 생성이 완료될 때까지 해당 테이블의 쓰기 작업(Update, Insert, Delete)이 차단되어 서비스 지연 및 가용성 저하가 발생할 수 있습니다.
서비스에 미치는 영향을 최소화하려면
CONCURRENTLY옵션을 사용하여 쓰기 락을 방지하는 것을 고려해 보세요.
단, Flyway와 같은 데이터베이스 마이그레이션 도구는 기본적으로 트랜잭션 내에서 스크립트를 실행합니다.CONCURRENTLY키워드는 트랜잭션 내부에서 사용할 수 없으므로, 이를 적용하려면 해당 마이그레이션 파일이 트랜잭션 없이(Non-transactional) 실행되도록 별도의 설정이나 파일명 변경이 필요할 수 있습니다.CREATE UNIQUE INDEX CONCURRENTLY uk_chat_members_active_room_user ON chat_members (room_id, user_id) WHERE status = 'ACTIVATE';🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@manabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sql` around lines 1 - 3, Update the uk_chat_members_active_room_user migration to create the partial unique index with the CONCURRENTLY option, and configure this migration to run non-transactionally as required by Flyway. Preserve the existing index name, columns, uniqueness, and status filter.Source: Linters/SAST tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In
`@manabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sql`:
- Around line 4-11: Update the chk_cancellation_request_status constraint in the
migration to add it with NOT VALID, then separately validate it using VALIDATE
CONSTRAINT so constraint creation does not scan and lock the entire table during
the initial ALTER TABLE operation.
In
`@manabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sql`:
- Around line 1-3: Update the uk_chat_members_active_room_user migration to
create the partial unique index with the CONCURRENTLY option, and configure this
migration to run non-transactionally as required by Flyway. Preserve the
existing index name, columns, uniqueness, and status filter.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: abf70e11-ce53-4f00-b90e-4e6999b03786
📒 Files selected for processing (9)
manabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatRoomService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationExpirationService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationService.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingCancellationRequestRepository.javamanabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sqlmanabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sqlmanabom/src/test/java/mannabom_server/manabom/application/chat/service/ChatRoomServiceTest.javamanabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationExpirationServiceTest.javamanabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationServiceTest.java
🚧 Files skipped from review as they are similar to previous changes (3)
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingCancellationRequestRepository.java
- manabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatRoomService.java
- manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationService.java
kimjuneon
requested changes
Jul 27, 2026
kimjuneon
left a comment
kimjuneon
left a comment
Contributor
There was a problem hiding this comment.
확인했습니다 아래 부분만 한번 확인해주시면 좋을 것 같아요
Sehi55
force-pushed
the
feat/meeting-cancellation
branch
from
August 6, 2026 05:03
fa62312 to
55f187b
Compare
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
There was a problem hiding this comment.
Actionable comments posted: 4
🧹 Nitpick comments (1)
manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql (1)
7-18: 🩺 Stability & Availability | 🔵 Trivial | 🏗️ Heavy lift온라인 migration 전략을 적용하십시오.
기본 제약 조건 검증과 인덱스 생성은 쓰기 잠금을 발생시킬 수 있습니다.
manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql#L7-L18: FK와 CHECK를NOT VALID로 추가한 뒤 별도 단계에서 검증하십시오.manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql#L20-L25: 인덱스에CONCURRENTLY사용을 검토하고 migration runner 설정을 확인하십시오.manabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sql#L4-L11: 상태 CHECK를NOT VALID로 추가한 뒤 별도 단계에서 검증하십시오.manabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sql#L1-L3: 유니크 인덱스에CONCURRENTLY사용을 검토하고 migration runner 설정을 확인하십시오.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql` around lines 7 - 18, Apply the online migration strategy across the specified migration sites: in manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql lines 7-18, add the foreign-key and check constraints as NOT VALID and validate them separately; in lines 20-25, use concurrent index creation where applicable and verify the migration runner supports it. In manabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sql lines 4-11, add the status check as NOT VALID and validate it in a separate step. In manabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sql lines 1-3, use a concurrently created unique index and confirm the migration runner’s transaction configuration supports concurrent DDL.Source: Linters/SAST tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql`:
- Around line 20-22: Before creating the unique index in
manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql
lines 20-22, remove duplicate PENDING rows sharing the same non-null
meeting_match_id, retaining one row per match. Apply the same cleanup in
manabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sql
lines 1-3 for duplicate ACTIVATE rows sharing (room_id, user_id), then create
each unique index.
- Around line 1-5: Update migration V22 around the meeting_cancellation_requests
schema changes to preserve existing pending requests: either keep legacy
meeting_id-based lookup support, or backfill meeting_match_id from a verified
meeting_matches relationship and update the related current, approval, and
rejection APIs to use it. Do not leave existing rows with null meeting_match_id
while only the new meeting_match_id path is active.
In
`@manabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sql`:
- Around line 1-11: 기존 WITHDRAWN 행을 처리하지 않아 chk_cancellation_request_status 추가가
실패할 수 있습니다. meeting_cancellation_requests의 WITHDRAWN 데이터를 제약 조건 제거 및 재추가 전에 적절한
유효 상태로 변환하거나 삭제하는 데이터 마이그레이션을 추가하고, 이후에는 기존 제약 조건 변경 흐름을 유지하십시오.
In
`@manabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationServiceTest.java`:
- Around line 46-57: Update the unanimous-approval setup in
MeetingCancellationServiceTest so the allMembersAgreed stub matches the
vote(30L, ...) call by stubbing it with meeting ID 30L instead of a null
argument. Add the chatRoomRepository mock required by the approval path before
verifying cancelByAgreement(), and keep the existing approval assertions
unchanged.
---
Nitpick comments:
In
`@manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql`:
- Around line 7-18: Apply the online migration strategy across the specified
migration sites: in
manabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sql
lines 7-18, add the foreign-key and check constraints as NOT VALID and validate
them separately; in lines 20-25, use concurrent index creation where applicable
and verify the migration runner supports it. In
manabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sql
lines 4-11, add the status check as NOT VALID and validate it in a separate
step. In
manabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sql
lines 1-3, use a concurrently created unique index and confirm the migration
runner’s transaction configuration supports concurrent DDL.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 11b0a866-6776-4ef7-8192-ced93998e404
📒 Files selected for processing (33)
manabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatRoomService.javamanabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/dto/request/MeetingCancellationVoteRequest.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/dto/response/MeetingCancellationResponse.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/dto/response/MeetingCancellationVoteResponse.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/scheduler/MeetingCancellationScheduler.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationExpirationService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingMatchingService.javamanabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingService.javamanabom/src/main/java/mannabom_server/manabom/domain/chat/repository/ChatMemberRepository.javamanabom/src/main/java/mannabom_server/manabom/domain/chat/repository/ChatRoomRepository.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/entity/Meeting.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/entity/MeetingCancellationRequest.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/entity/MeetingCancellationVote.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/CancellationVoteDecision.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/MeetingCancellationStatus.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/MeetingStatus.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingCancellationRequestRepository.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingCancellationVoteRepository.javamanabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingMatchRepository.javamanabom/src/main/java/mannabom_server/manabom/global/error/MeetingCancellationExpiredException.javamanabom/src/main/java/mannabom_server/manabom/infrastructure/security/websocket/StompAuthChannelInterceptor.javamanabom/src/main/java/mannabom_server/manabom/presentation/meeting/controller/MeetingController.javamanabom/src/main/resources/db/migration/V21__add_meeting_cancellation_tables.sqlmanabom/src/main/resources/db/migration/V22__move_meeting_cancellation_to_match.sqlmanabom/src/main/resources/db/migration/V23__remove_withdrawn_cancellation_status.sqlmanabom/src/main/resources/db/migration/V24__add_unique_active_chat_member.sqlmanabom/src/test/java/mannabom_server/manabom/application/chat/service/ChatRoomServiceTest.javamanabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationExpirationServiceTest.javamanabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationServiceTest.javamanabom/src/test/java/mannabom_server/manabom/domain/meeting/MeetingCancellationDomainTest.javamanabom/src/test/java/mannabom_server/manabom/domain/meeting/MeetingMemberLeaveStatusTest.java
🚧 Files skipped from review as they are similar to previous changes (27)
- manabom/src/main/java/mannabom_server/manabom/application/meeting/scheduler/MeetingCancellationScheduler.java
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/MeetingCancellationStatus.java
- manabom/src/main/java/mannabom_server/manabom/infrastructure/security/websocket/StompAuthChannelInterceptor.java
- manabom/src/main/java/mannabom_server/manabom/domain/chat/repository/ChatRoomRepository.java
- manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationExpirationService.java
- manabom/src/test/java/mannabom_server/manabom/domain/meeting/MeetingMemberLeaveStatusTest.java
- manabom/src/main/java/mannabom_server/manabom/application/meeting/dto/response/MeetingCancellationResponse.java
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/CancellationVoteDecision.java
- manabom/src/main/java/mannabom_server/manabom/application/meeting/dto/request/MeetingCancellationVoteRequest.java
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingCancellationVoteRepository.java
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingCancellationRequestRepository.java
- manabom/src/main/resources/db/migration/V21__add_meeting_cancellation_tables.sql
- manabom/src/main/java/mannabom_server/manabom/application/meeting/dto/response/MeetingCancellationVoteResponse.java
- manabom/src/test/java/mannabom_server/manabom/domain/meeting/MeetingCancellationDomainTest.java
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/repository/MeetingMatchRepository.java
- manabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatService.java
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/entity/MeetingCancellationVote.java
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/enums/MeetingStatus.java
- manabom/src/test/java/mannabom_server/manabom/application/chat/service/ChatRoomServiceTest.java
- manabom/src/main/java/mannabom_server/manabom/domain/chat/repository/ChatMemberRepository.java
- manabom/src/main/java/mannabom_server/manabom/domain/meeting/entity/MeetingCancellationRequest.java
- manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingMatchingService.java
- manabom/src/main/java/mannabom_server/manabom/presentation/meeting/controller/MeetingController.java
- manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingService.java
- manabom/src/main/java/mannabom_server/manabom/application/chat/service/ChatRoomService.java
- manabom/src/main/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationService.java
- manabom/src/test/java/mannabom_server/manabom/application/meeting/service/MeetingCancellationExpirationServiceTest.java
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
변경 사항
Summary by CodeRabbit