Skip to content

feat: 게스트 목록 조회 및 게스트 수 조회 추가 - #194

Merged
Kohseoyoung merged 1 commit into
mainfrom
feature/193-guest-list
Aug 13, 2026
Merged

feat: 게스트 목록 조회 및 게스트 수 조회 추가 #194
Kohseoyoung merged 1 commit into
mainfrom
feature/193-guest-list

Conversation

@Kohseoyoung

@Kohseoyoung Kohseoyoung commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

🔗 관련 이슈

Closes #193

📝 작업 내용

공유 링크로 참여한 게스트를 프로젝트 멤버가 조회할 수 있는 관리자용 API를 추가했습니다.

추가

  • GET /videos/{videoId}/share-links/{shareLinkId}/guests — 게스트 목록 조회
    • 응답: guestId, name, createdAt (세션 토큰 등 민감정보 제외)
    • 프로젝트 활성 멤버만 조회 가능
    • 경로의 videoId와 링크의 영상이 다르면 404
  • ShareLinkInfoResDTOguestCount 추가
  • GuestRepository에 목록·카운트 쿼리 추가

리팩터 (동작 변화 없음)

  • 멤버 검증·영상 조회 중복 로직을 private 메서드(validateProjectMember, findVideoOrThrow)로 추출

✅ 체크리스트

  • PR 제목은 커밋 컨벤션을 따랐습니다.
  • 관련 이슈를 연결했습니다.
  • 변경 사항에 대한 테스트를 진행했습니다.

Summary by CodeRabbit

  • 새 기능

    • 공유 링크 소유자가 참여 중인 게스트 목록을 조회할 수 있습니다.
    • 공유 링크 조회 결과에 현재 게스트 수가 표시됩니다.
    • 게스트 목록에는 이름, 식별자, 참여 시각이 제공되며 민감한 세션 정보는 포함되지 않습니다.
  • 버그 수정 및 검증

    • 공유 링크 관련 요청의 영상 및 링크 식별자 유효성 검증이 강화되었습니다.
    • 영상과 공유 링크의 일치 여부 및 접근 권한을 확인합니다.
  • 문서

    • API 예시 이름이 최신 내용에 맞게 업데이트되었습니다.

    부수 수정

  • FeedbackResponse Swagger example 값 정정 (문서 표기만, 동작 변화 없음)

@Kohseoyoung Kohseoyoung self-assigned this Aug 13, 2026
@Kohseoyoung Kohseoyoung added the feature 새로운 기능 추가 label Aug 13, 2026
@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0c9d1b0a-d796-4b2e-aa5c-1417247dec37

📥 Commits

Reviewing files that changed from the base of the PR and between 70a32f5 and ae58595.

📒 Files selected for processing (6)
  • src/main/java/com/slatto/domain/feedback/dto/response/FeedbackResponse.java
  • src/main/java/com/slatto/domain/sharelink/controller/ShareLinkController.java
  • src/main/java/com/slatto/domain/sharelink/converter/ShareLinkConverter.java
  • src/main/java/com/slatto/domain/sharelink/dto/response/ShareLinkResponse.java
  • src/main/java/com/slatto/domain/sharelink/repository/GuestRepository.java
  • src/main/java/com/slatto/domain/sharelink/service/ShareLinkService.java

📝 Walkthrough

Walkthrough

공유 링크의 게스트 수와 게스트 목록 조회 기능을 추가했습니다. 프로젝트 활성 멤버 권한과 영상·공유 링크 연결을 검증합니다. 게스트 응답에는 ID, 이름, 참여 시각만 포함합니다. 기존 엔드포인트의 경로 변수 검증도 보강했습니다.

Changes

공유 링크 게스트 조회

Layer / File(s) Summary
게스트 응답 계약 및 조회 데이터
src/main/java/com/slatto/domain/sharelink/dto/response/ShareLinkResponse.java, src/main/java/com/slatto/domain/sharelink/repository/GuestRepository.java, src/main/java/com/slatto/domain/sharelink/converter/ShareLinkConverter.java, src/main/java/com/slatto/domain/feedback/dto/response/FeedbackResponse.java
게스트 요약 응답과 guestCount 필드를 추가했습니다. 게스트 목록은 생성 시각 역순으로 조회합니다. 세션 토큰은 응답에서 제외합니다. Swagger 예시 이름을 변경했습니다.
게스트 조회 서비스 흐름
src/main/java/com/slatto/domain/sharelink/service/ShareLinkService.java
영상과 공유 링크의 연결을 확인합니다. 프로젝트 활성 멤버 권한을 확인합니다. 공유 링크 정보에 게스트 수를 포함하고 게스트 목록을 반환합니다.
게스트 조회 API 연결
src/main/java/com/slatto/domain/sharelink/controller/ShareLinkController.java
GET /videos/{videoId}/share-links/{shareLinkId}/guests 엔드포인트를 추가했습니다. 기존 조회 및 토글 API의 경로 변수에 @Positive 검증을 적용했습니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Mergeability Score: ⚪ Minimal · up to ae585

The guest listing and count additions do not show a concrete correctness or production risk in the supplied evidence; no actionable merge-blocking risk remains after normal checks and review.

Sequence Diagram(s)

sequenceDiagram
  participant 프로젝트멤버
  participant ShareLinkController
  participant ShareLinkService
  participant GuestRepository
  프로젝트멤버->>ShareLinkController: GET 게스트 목록 요청
  ShareLinkController->>ShareLinkService: getGuests(videoId, shareLinkId, userId)
  ShareLinkService->>GuestRepository: 공유 링크 게스트 목록 조회
  GuestRepository-->>ShareLinkService: 생성 시각 역순 게스트 목록
  ShareLinkService-->>ShareLinkController: GuestListResDTO
  ShareLinkController-->>프로젝트멤버: 게스트 ID, 이름, 참여 시각 반환
Loading

Possibly related PRs

  • SLAT-TO/SLATE-TO-BE#60: FeedbackResponse.ActorDTO의 Swagger 이름 예시를 함께 변경합니다.
  • SLAT-TO/SLATE-TO-BE#164: ShareLinkService, ShareLinkConverter, ShareLinkResponse의 게스트 처리 변경과 연결됩니다.
  • SLAT-TO/SLATE-TO-BE#179: ShareLinkControllerShareLinkService의 게스트 관련 API 변경과 연결됩니다.

Suggested labels: refactor

Suggested reviewers: chazy-d, sangwon02, guingguing

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning FeedbackResponse.java의 Swagger 예시 변경은 연결된 이슈의 게스트 조회 기능과 직접 관련이 없습니다. FeedbackResponse.java의 예시 값 변경을 제거하거나 해당 변경이 필요한 이유를 PR 범위에 명시하세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 38.89% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed 게스트 목록 API, 활성 멤버 검증, 민감정보 제외, guestCount 및 저장소 쿼리 요구 사항을 구현했습니다.
Title check ✅ Passed 제목이 게스트 목록 조회와 게스트 수 조회라는 PR의 핵심 변경 사항을 명확하게 요약합니다.
Description check ✅ Passed 관련 이슈, 작업 내용, 주요 동작, 제외 범위, 체크리스트를 모두 포함해 템플릿 요구 사항을 충족합니다.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch

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.

@Kohseoyoung
Kohseoyoung merged commit 831cdff into main Aug 13, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FEAT: 게스트 목록 조회 및 게스트 수 조회

1 participant