Skip to content

Hotfix : 마이페이지에서 참여중, 호스트 파티 조회시 최신순으로 조회되도록 수정#99

Merged
milowon merged 68 commits intomainfrom
develop
Jan 10, 2026
Merged

Hotfix : 마이페이지에서 참여중, 호스트 파티 조회시 최신순으로 조회되도록 수정#99
milowon merged 68 commits intomainfrom
develop

Conversation

@milowon
Copy link
Contributor

@milowon milowon commented Jan 10, 2026

📝 상세 내용

  • 마이페이지에서 참여중, 호스트 파티 조회시 최신순으로 조회되도록 수정

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 파티 참여 요청 알림 추가
    • 파티 완료 알림 추가
  • 개선사항

    • 파티 승인 및 거절 알림 메시지 개선
    • 파티 목록을 최신순으로 정렬 적용
  • 변경사항

    • 기존 API 엔드포인트 제거 및 정리

✏️ Tip: You can customize this high-level summary in your review settings.

yyytir777 and others added 30 commits December 8, 2025 10:50
* feat : google login 구현 완료 (ios 구현 중)
* feat : google login 구현 완료
* fix : user hard delete
* feat : apple 로그인 구현 및 ddl-auto -> update 변경
* 약관 엔티티 생성 및 연관관계 설정

* 회원가입에 약관 저장 로직 추가

* 서버에서 idToken을 받아올 수 없으므로 단순히 이메일로 accessToken을 받아오는 test API 추가
* feat : 파티 엔티티 정의

* feat : 파티 dto

* feat : party dto 정의

* feat : party entity 정의

* feat : 파티 생성,수정,삭제, 조회

partycontroller
partyservice
partyrepository

* feat : 거리 계산 클래스

* refactor : 불필요한 코드 삭제

* refactor : token provider로 유저 아이디 추출하도록 변경

* Fix: 파티 기능 버그 수정

* docs : 파티 swagger 문서 추가
milowon and others added 24 commits January 3, 2026 09:46
* feat : 파티 검색 추가
* feature & fix : 유저 최근 검색어 API 구현
* feat : stomp import 및 인증 설정 진행

* feat : 웹소켓 subscribe, unsubscribe, disconnect 시 유저의 실시간 채팅방 접속 정보 수정

* feat : chatMessage 엔티티 추가

* feat : 채팅 전송, 채팅이력 조회 API 생성

* fix : 채팅 기능 고도화

* fix : minor change

* fix : 웹소켓에서 오류 발생 시 에러 메시지 사용자에게 보냄

* fix : 참여 요청 거절 수정
* feat : 유저 프로필 이미지 수정, 삭제

* fix : 유저 프로필 수정, 삭제
* feat : 거리 계산 쿼리 PartySearchRepository로 분리 (거리순, 페이징, 검색)

* feat : 거리 계산 API 구현
Co-authored-by: marshmallowing <yuin1111801@naver.com>
@milowon milowon self-assigned this Jan 10, 2026
@milowon milowon added the bug Something isn't working label Jan 10, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 10, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

이 PR은 파티 알림 시스템을 재구성하여 파티 참여 요청, 승인, 거절, 자동 종료 및 완료 흐름을 중심으로 변경합니다. PartyNotificationService의 메서드 서명을 업데이트하여 partyTitle을 포함하고, NotificationFacade에 PartyRepository 및 UserRepository를 추가하여 파티 정보를 조회합니다. PartyService에 NotificationFacade를 통합하여 파티 참여, 승인, 거절 및 정산 시점에 알림을 발송합니다. PartyParticipantRepository에 새로운 쿼리 메서드를 추가하고, UserController에서 getActiveParties 엔드포인트를 제거하며, UserService의 정렬 로직을 개선합니다. 또한 PartyErrorCode 예외 정의를 신규 추가합니다.

Sequence Diagram

sequenceDiagram
    participant User as User
    participant PartyService as PartyService
    participant PartyRepository as PartyRepository
    participant NotificationFacade as NotificationFacade
    participant PartyNotificationService as PartyNotificationService
    participant FCMService as FCM Service

    User->>PartyService: joinParty(partyId, userId)
    PartyService->>PartyRepository: save(PartyParticipant)
    PartyService->>NotificationFacade: notifyNewPartyRequest(hostId, userId, partyId)
    NotificationFacade->>PartyRepository: findById(partyId)
    NotificationFacade->>PartyNotificationService: sendNewPartyRequestNotification(hostId, nickname, title, partyId)
    PartyNotificationService->>FCMService: send notification

    User->>PartyService: approveParticipant(participantId)
    PartyService->>NotificationFacade: notifyApproval(userId, partyId)
    NotificationFacade->>PartyRepository: findById(partyId)
    NotificationFacade->>PartyNotificationService: sendApprovalNotification(userId, title, partyId)
    PartyNotificationService->>FCMService: send notification

    User->>PartyService: settleParty(partyId)
    PartyService->>PartyRepository: findApprovedMembers(partyId)
    PartyService->>NotificationFacade: notifyPartyComplete(memberIds, partyId)
    NotificationFacade->>PartyRepository: findById(partyId)
    NotificationFacade->>PartyNotificationService: sendPartyCompleteNotification(memberIds, title, partyId)
    PartyNotificationService->>FCMService: send notification to all members
Loading

Possibly related PRs

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning PR 제목은 마이페이지의 참여중/호스트 파티 조회 시 최신순 정렬이라는 주요 변경사항을 명확하게 설명하고 있습니다. 그러나 실제 변경사항의 범위가 더 넓습니다. PR 제목은 주로 UserService의 정렬 로직 변경만 언급하고 있으나, 실제로는 NotificationFacade, PartyService, PartyController, 새로운 repository 등 광범위한 변경이 포함되어 있습니다. 더 포괄적인 제목으로 수정하거나 변경사항을 명확히 구분해야 합니다.
Docstring Coverage ⚠️ Warning Docstring coverage is 28.89% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 386563b and cf0f51d.

📒 Files selected for processing (7)
  • src/main/java/ita/tinybite/domain/notification/service/PartyNotificationService.java
  • src/main/java/ita/tinybite/domain/notification/service/facade/NotificationFacade.java
  • src/main/java/ita/tinybite/domain/party/repository/PartyParticipantRepository.java
  • src/main/java/ita/tinybite/domain/party/service/PartyService.java
  • src/main/java/ita/tinybite/domain/user/controller/UserController.java
  • src/main/java/ita/tinybite/domain/user/service/UserService.java
  • src/main/java/ita/tinybite/global/exception/errorcode/PartyErrorCode.java

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants