모임 상세 조회 시 내 정보 보이도록 다시 수정 by LimdaeIl · Pull Request #210 · WeGo-Together/WeGo_BackEnd

LimdaeIl · 2026-01-05T05:04:32Z

📝 Pull Request

📌 PR 종류

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

✨ 변경 내용

그룹 사용자 쿼리를 리팩토링하여 참석 사용자와 현재 사용자를 효율적으로 검색합니다.
이를 통해 현재 사용자가 명시적으로 참석으로 표시되지 않더라도 그룹 사용자 목록에 항상 포함되므로, 관련 정보를 표시하여 사용자 경험을 개선합니다.

🔍 관련 이슈

Closes [FEAT] 모임 상세 조회 시 내 정보 보이도록 다시 수정 #209

🧪 테스트

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

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

🚨 확인해야 할 사항 (Checklist)

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

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

🙋 기타 참고 사항

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

Summary by CodeRabbit

릴리스 노트

새로운 기능
- 그룹 상세 조회 시 참석자 목록에 요청 사용자가 포함되도록 개선되었습니다.
테스트
- 그룹 참석 승인 관련 테스트 케이스가 추가되었습니다.

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

그룹 사용자 쿼리를 리팩토링하여 참석 사용자와 현재 사용자를 효율적으로 검색합니다. 이를 통해 현재 사용자가 명시적으로 참석으로 표시되지 않더라도 그룹 사용자 목록에 항상 포함되므로, 관련 정보를 표시하여 사용자 경험을 개선합니다.

coderabbitai · 2026-01-05T05:04:44Z

Caution

Review failed

The pull request is closed.

Walkthrough

모임 상세 조회 시 비호스트 사용자에 대해 저장소 메서드를 변경했습니다. 기존 findAttendByGroupIdOrderByJoinedAtAscWithUser 호출 대신 새로운 findAttendPlusMeByGroupId 메서드를 사용하여 요청 사용자의 ID를 파라미터로 전달하고, 해당 메서드를 저장소에 추가했습니다.

Changes

Cohort / File(s)	변경 사항
GroupV2Service 로직 변경 `src/main/java/team/wego/wegobackend/group/v2/application/service/GroupV2Service.java`	GetGroupV2Response 흐름에서 비호스트 사용자 경로: `groupUserV2Repository.findAttendByGroupIdOrderByJoinedAtAscWithUser(groupId)` → `groupUserV2Repository.findAttendPlusMeByGroupId(groupId, userId)` 변경
GroupUserV2Repository 메서드 추가 `src/main/java/team/wego/wegobackend/group/v2/domain/repository/GroupUserV2Repository.java`	새 메서드 `findAttendPlusMeByGroupId(Long groupId, Long userId)` 추가: 상태가 ATTEND이거나 userId가 일치하는 GroupUserV2 레코드를 조회하고 joinedAt 오름차순으로 정렬
HTTP 테스트 업데이트 `src/test/http/group/v2/v2-group-attend-approval-required.http`	MEMBER2 참여 테스트 단계 추가: POST /api/v2/groups/{{groupId_approvalTest}}/attend 호출 후 그룹 상세 조회 시 host2AccessToken 사용으로 변경

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

[FEAT] HOST가 모임 상세 조회 응답 필드 수정 #181: 동일하게 GetGroupV2Response의 멤버 선택 로직을 수정하여 비호스트 뷰어의 멤버 필터링을 변경하므로 같은 코드 경로를 수정합니다.
모임 참여 카운트와 정렬 순서 개선 #206: GroupV2Service와 GroupUserV2Repository의 그룹 참가자 조회 로직을 수정하여, 호스트/비호스트 분기와 기존 저장소 메서드를 구현했으며, 본 PR이 이를 확장합니다.
[FEAT] 모임 참여와 취소 기능 구현 #131: GroupV2Service의 GetGroupV2Response와 GroupUserV2Repository의 참석/멤버십 관련 메서드 수정으로 동일 서비스/저장소 로직을 다루고 있습니다.

Poem

🐰 모임 상세를 다시 들여다보니,
내 정보가 쏙 들어왔어요!
ATTEND든 아니든 나를 포함해,
저장소 메서드 findAttendPlusMe 하나로
요청자의 마음을 담아냅니다! 🌟

✨ 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 9bb2ecb and bbb5367.

📒 Files selected for processing (3)

src/main/java/team/wego/wegobackend/group/v2/application/service/GroupV2Service.java
src/main/java/team/wego/wegobackend/group/v2/domain/repository/GroupUserV2Repository.java
src/test/http/group/v2/v2-group-attend-approval-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.}

Copilot

Pull request overview

This PR modifies the group detail retrieval logic to ensure the current user's information is always visible, even when they are not explicitly marked as attending. The change refactors the group user query to fetch both attending users and the current user efficiently, improving the user experience by displaying relevant information consistently.

Key Changes:

Added a new repository method to fetch attending users plus the current user
Updated the service layer to use this new query method for non-host users
Added test case for a second member's participation

Reviewed changes

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

File	Description
GroupUserV2Repository.java	Added `findAttendPlusMeByGroupId` method to retrieve attending users plus the current user
GroupV2Service.java	Updated group detail retrieval to use the new repository method for non-host users
v2-group-attend-approval-required.http	Added test case for member2 participation and updated authorization token in test case 7

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

Copilot · 2026-01-05T05:08:02Z

src/main/java/team/wego/wegobackend/group/v2/domain/repository/GroupUserV2Repository.java

+                  and (gu.status = team.wego.wegobackend.group.v2.domain.entity.GroupUserV2Status.ATTEND
+                       or u.id = :userId)


The query may return duplicate results if a user has ATTEND status AND matches the userId condition. Consider adding DISTINCT to the query or handling duplicates in the service layer.

Copilot · 2026-01-05T05:08:02Z

src/main/java/team/wego/wegobackend/group/v2/domain/repository/GroupUserV2Repository.java

+    List<GroupUserV2> findAttendPlusMeByGroupId(@Param("groupId") Long groupId,
+            @Param("userId") Long userId);


The method name findAttendPlusMeByGroupId doesn't clearly indicate that it may return users with non-ATTEND status. Consider renaming to findAttendingUsersAndCurrentUser or findAttendingUsersIncludingUser for better clarity.

Suggested change

List<GroupUserV2> findAttendPlusMeByGroupId(@Param("groupId") Long groupId,

@Param("userId") Long userId);

List<GroupUserV2> findAttendingUsersAndCurrentUser(@Param("groupId") Long groupId,

@Param("userId") Long userId);

@Deprecated

default List<GroupUserV2> findAttendPlusMeByGroupId(@Param("groupId") Long groupId,

@Param("userId") Long userId) {

return findAttendingUsersAndCurrentUser(groupId, userId);

}

Copilot · 2026-01-05T05:08:02Z

src/test/http/group/v2/v2-group-attend-approval-required.http

 ### 7. (선택) 모임 상세 조회 - MEMBER1 토큰으로 내 상태 확인
 GET http://localhost:8080/api/v2/groups/{{groupId_approvalTest}}
-Authorization: Bearer {{member1AccessToken}}
+Authorization: Bearer {{host2AccessToken}}


The test description says 'MEMBER1 토큰으로 내 상태 확인' (check my status with MEMBER1 token) but uses host2AccessToken instead. Update the comment to match the actual token being used or revert to member1AccessToken if the comment is correct.

Suggested change

Authorization: Bearer {{host2AccessToken}}

Authorization: Bearer {{member1AccessToken}}

LimdaeIl self-assigned this Jan 5, 2026

Copilot AI review requested due to automatic review settings January 5, 2026 05:04

LimdaeIl added this to WeGo-Together Backend Jan 5, 2026

LimdaeIl added the ✨enhancement New feature or request label Jan 5, 2026

github-project-automation bot moved this to Backlog in WeGo-Together Backend Jan 5, 2026

LimdaeIl moved this from Backlog to In progress in WeGo-Together Backend Jan 5, 2026

Copilot started reviewing on behalf of LimdaeIl January 5, 2026 05:04 View session

LimdaeIl merged commit 2297e6a into main Jan 5, 2026
4 of 5 checks passed

LimdaeIl deleted the feat/group-detail branch January 5, 2026 05:04

github-project-automation bot moved this from In progress to Done in WeGo-Together Backend Jan 5, 2026

Copilot AI reviewed Jan 5, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

모임 상세 조회 시 내 정보 보이도록 다시 수정 #210

모임 상세 조회 시 내 정보 보이도록 다시 수정 #210
LimdaeIl merged 1 commit intomainfrom
feat/group-detail

LimdaeIl commented Jan 5, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Jan 5, 2026 •

edited

Loading

Review failed

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jan 5, 2026

Uh oh!

Copilot AI Jan 5, 2026

Uh oh!

Copilot AI Jan 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

		and (gu.status = team.wego.wegobackend.group.v2.domain.entity.GroupUserV2Status.ATTEND
		or u.id = :userId)

		List<GroupUserV2> findAttendPlusMeByGroupId(@Param("groupId") Long groupId,
		@Param("userId") Long userId);

-    List<GroupUserV2> findAttendPlusMeByGroupId(@Param("groupId") Long groupId,
-            @Param("userId") Long userId);
+    List<GroupUserV2> findAttendingUsersAndCurrentUser(@Param("groupId") Long groupId,
+            @Param("userId") Long userId);
+    @Deprecated
+    default List<GroupUserV2> findAttendPlusMeByGroupId(@Param("groupId") Long groupId,
+            @Param("userId") Long userId) {
+        return findAttendingUsersAndCurrentUser(groupId, userId);
+    }

	Authorization: Bearer {{host2AccessToken}}
	Authorization: Bearer {{member1AccessToken}}

Conversation

LimdaeIl commented Jan 5, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📝 Pull Request

📌 PR 종류

✨ 변경 내용

🔍 관련 이슈

🧪 테스트

🚨 확인해야 할 사항 (Checklist)

🙋 기타 참고 사항

Summary by CodeRabbit

릴리스 노트

Uh oh!

coderabbitai bot commented Jan 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

LimdaeIl commented Jan 5, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jan 5, 2026 •

edited

Loading