Skip to content

[FEAT] 모임 신청 메시지 갱신 수정#197

Merged
LimdaeIl merged 1 commit intomainfrom
feat/group-attend
Jan 2, 2026
Merged

[FEAT] 모임 신청 메시지 갱신 수정#197
LimdaeIl merged 1 commit intomainfrom
feat/group-attend

Conversation

@LimdaeIl
Copy link
Collaborator

@LimdaeIl LimdaeIl commented Jan 2, 2026

📝 Pull Request

📌 PR 종류

해당하는 항목에 체크해주세요.

  • 기능 추가 (Feature)
  • 버그 수정 (Fix)
  • 문서 수정 (Docs)
  • 코드 리팩터링 (Refactor)
  • 테스트 추가 (Test)
  • 기타 변경 (Chore)

✨ 변경 내용

그룹 가입 요청 처리 과정에 메시지를 추가하도록 수정했습니다.

이를 통해 사용자는 그룹 가입 요청 시 추가 정보를 제공할 수 있습니다.

GroupUserV2 클래스의 requestJoin 메서드가 이제 메시지 매개변수를 받습니다.

받은 메시지는 가입 ​​요청과 연결됩니다.

GroupUserV2 클래스의 createPending 메서드 또한 이제 메시지 매개변수를 받습니다.

GroupV2AttendanceService가 그룹 생성 또는 가입 요청 시 메시지를 제공하도록 업데이트되었습니다.

테스트 HTTP 메서드도 테스트 메시지를 포함하도록 업데이트되었습니다.

🔍 관련 이슈

🧪 테스트

변경된 기능에 대한 테스트 범위 또는 테스트 결과를 작성해주세요.

  • 유닛 테스트 추가 / 수정
  • 통합 테스트 검증
  • 수동 테스트 완료

🚨 확인해야 할 사항 (Checklist)

PR을 제출하기 전에 아래 항목들을 확인해주세요.

  • 코드 포매팅 완료
  • 불필요한 파일/코드 제거
  • 로직 검증 완료
  • 프로젝트 빌드 성공
  • 린트/정적 분석 통과 (해당 시)

🙋 기타 참고 사항

리뷰어가 참고하면 좋을 만한 추가 설명이 있다면 적어주세요.

Summary by CodeRabbit

릴리스 노트

  • 신기능
    • 승인 필수 그룹 참여 신청 시 메시지를 함께 전달할 수 있도록 개선되었습니다.

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

그룹 가입 요청 처리 과정에 메시지를 추가하도록 수정했습니다.

이를 통해 사용자는 그룹 가입 요청 시 추가 정보를 제공할 수 있습니다.

`GroupUserV2` 클래스의 `requestJoin` 메서드가 이제 메시지 매개변수를 받습니다.

받은 메시지는 가입 ​​요청과 연결됩니다.

`GroupUserV2` 클래스의 `createPending` 메서드 또한 이제 메시지 매개변수를 받습니다.

GroupV2AttendanceService가 그룹 생성 또는 가입 요청 시 메시지를 제공하도록 업데이트되었습니다.

테스트 HTTP 메서드도 테스트 메시지를 포함하도록 업데이트되었습니다.
@LimdaeIl LimdaeIl self-assigned this Jan 2, 2026
Copilot AI review requested due to automatic review settings January 2, 2026 03:19
@LimdaeIl LimdaeIl added the ✨enhancement New feature or request label Jan 2, 2026
@LimdaeIl LimdaeIl moved this from Backlog to In progress in WeGo-Together Backend Jan 2, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 2, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

메시지 매개변수를 추가하여 GroupUserV2.requestJoin() 메서드 시그니처를 업데이트했습니다. 모임 신청 시 사용자가 제공한 메시지를 함께 저장하도록 변경되었으며, 테스트 시나리오도 새로운 메시지 필드를 포함하도록 수정되었습니다.

Changes

코호트 / 파일 변경 사항
메서드 시그니처 및 로직 변경
src/main/java/team/wego/wegobackend/group/v2/domain/entity/GroupUserV2.java
public void requestJoin()public void requestJoin(String message)로 변경. 새 메서드는 PENDING 상태 설정, 타임스탬프 업데이트, 제공된 메시지를 적용하도록 구현
서비스 레이어 호출 업데이트
src/main/java/team/wego/wegobackend/group/v2/application/service/GroupV2AttendanceService.java
APPROVAL_REQUIRED 조건에서 기존 멤버의 재신청 처리 시 groupUserV2.requestJoin(message)로 변경하여 메시지 전달
테스트 시나리오 및 요청 본문 추가
src/test/http/group/v2/notification/v2-group-notification-approve_required.http
FOLLOWER1, FOLLOWER2의 attend 요청에 { "message": "..." } JSON 본문 추가. 새로운 재신청 테스트 시나리오(3-2-1) 추가

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • [FEAT] 모임 PENDING 목록 조회 구현 #174: 본 PR의 requestJoin(String) 메서드 추가 및 메시지 전달 기능은 PR #174에서 도입된 joinRequestMessage 필드, requestJoin() 오버로드, AttendGroupV2Request DTO, 그리고 메시지를 전파하는 컨트롤러/서비스 변경과 직접적으로 연관됨

Poem

🐰 메시지 소리 삐악삐악,
신청할 때 함께 담으니,
모임의 마음 이제 보여,
재신청도 말맺음 하고,
우리 함께, 더 잘 들어요! 🎉

✨ 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 5ddf063 and 4342141.

📒 Files selected for processing (3)
  • src/main/java/team/wego/wegobackend/group/v2/application/service/GroupV2AttendanceService.java
  • src/main/java/team/wego/wegobackend/group/v2/domain/entity/GroupUserV2.java
  • src/test/http/group/v2/notification/v2-group-notification-approve_required.http

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.

@LimdaeIl LimdaeIl merged commit 10c4655 into main Jan 2, 2026
4 of 5 checks passed
@LimdaeIl LimdaeIl deleted the feat/group-attend branch January 2, 2026 03:20
@github-project-automation github-project-automation bot moved this from In progress to Done in WeGo-Together Backend Jan 2, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the group join request functionality by adding support for optional messages when users apply to join a group. The primary change consolidates duplicate requestJoin methods in the GroupUserV2 class and ensures the message parameter is properly propagated through the system.

  • Refactored GroupUserV2.requestJoin() to accept a message parameter, removing method duplication
  • Updated GroupV2AttendanceService to pass the message parameter when handling re-join requests
  • Enhanced test HTTP files with message payloads to verify the new functionality

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/main/java/team/wego/wegobackend/group/v2/domain/entity/GroupUserV2.java Consolidated two requestJoin methods into a single method accepting a message parameter, eliminating code duplication
src/main/java/team/wego/wegobackend/group/v2/application/service/GroupV2AttendanceService.java Updated re-join flow to pass message parameter to requestJoin method
src/test/http/group/v2/notification/v2-group-notification-approve_required.http Added message payloads to test requests for join and re-join scenarios

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Status: Done

Development

Successfully merging this pull request may close these issues.

[FEAT] 모임 신청 메시지 갱신 수정

1 participant