Skip to content

후기 모달 안 뜨는 부분 수정#450

Merged
HA-SEUNG-JEONG merged 6 commits into
mainfrom
hotfix/study-review
Mar 25, 2026
Merged

후기 모달 안 뜨는 부분 수정#450
HA-SEUNG-JEONG merged 6 commits into
mainfrom
hotfix/study-review

Conversation

@HA-SEUNG-JEONG

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented Mar 25, 2026

Copy link
Copy Markdown
Contributor

🌱 연관된 이슈

☘️ 작업 내용

🍀 참고사항

스크린샷 (선택)

Summary by CodeRabbit

릴리스 노트

  • New Features

    • 그룹 스터디 리뷰 모달 자동 표시 기능 추가
    • 스터디 완료 축하 애니메이션 추가
    • 탭 잠금 상태 및 안내 툴팁 기능 추가
  • Bug Fixes

    • 시각적 레이어링 우선순위 조정
    • 중복 import 정리
  • Refactor

    • 인증 상태 기반 동적 데이터 처리 개선
    • 리뷰 작성 여부 자동 감지 로직 추가
    • 모달 조건부 렌더링 최적화
  • Style

    • 탭 컨테이너 가로 스크롤 기능 추가
    • 색상 토큰 기반 스타일링 개선

@HA-SEUNG-JEONG HA-SEUNG-JEONG self-assigned this Mar 25, 2026
@vercel

vercel Bot commented Mar 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
study-platform-client-dev Error Error Mar 26, 2026 0:57am

@coderabbitai

coderabbitai Bot commented Mar 25, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

개요

이 PR은 그룹 스터디 리뷰 기능을 추가하며, 리뷰 모달의 자동 열기 동작, 7일 검토 창 제한 로직, 그룹 스터디 상세 페이지의 탭 잠금 처리를 구현합니다. 또한 memberId 파생, 모달 조건부 렌더링 재구성, 탭 컴포넌트 토론 툴팁 지원이 포함됩니다.

변경사항

그룹 / 파일(들) 요약
그룹 리뷰 카드 임포트
src/app/(service)/(my)/my-study-review/group/[groupStudyId]/page.tsx
GroupReviewCard 임포트 중복 제거 및 재정렬.
회원 스터디 목록 쿼리 업데이트
src/app/(service)/(my)/my-study/completed/page.tsx, src/app/(service)/(my)/my-study/not-completed/page.tsx
useAuthReady()에서 memberId 파생으로 변경하고 쿼리 입력을 memberId ?? 0으로 업데이트.
모달 Props 타입 좁히기
src/components/common/modals/group-study-review-modal.tsx
GroupStudyReviewModalPropsdetailInfobasicInfo를 필요한 필드만 포함하는 Pick 타입으로 제한.
완료 모달 z-index 조정
src/components/common/modals/study-completion-modal.tsx
캔버스 요소의 z-index를 45에서 55로 증가.
탭 컴포넌트 리팩터링
src/components/common/ui/tabs/index.tsx
lockedTooltip 옵션 추가, 잠금된 탭 렌더링 로직 구현, 가로 스크롤 지원, 탭 스타일 기본 클래스 공유화.
그룹 스터디 상세 페이지 리뷰 모달 추가
src/components/pages/group-study-detail-page.tsx
동적 임포트 모달 추가, 검토 창 제한 로직, 리뷰 작성 여부 쿼리, useRef 기반 일회 자동 열기, 탭 잠금 설정.
프리미엄 스터디 상세 페이지 리뷰 모달 추가
src/components/pages/premium-study-detail-page.tsx
동적 임포트 모달, 검토 창 제한 로직, 일회 자동 열기 동작, 모달 흐름 연결.
목록 페이지 모달 조건부 렌더링 재구성
src/components/pages/group-study-list-page.tsx, src/components/pages/premium-study-list-page.tsx
GroupStudyReviewModal은 조건부 렌더링, StudyCompletionModal은 무조건 마운트로 변경.
리뷰 미리알림 훅 업데이트
src/hooks/common/use-group-study-review-reminder.ts
isWithinReviewAvailableWindow 내보내기 추가, 검토 창 로직 수정, 쿼리 완료 감지 방식 변경, DTO 타입 단언 제거.

예상 코드 리뷰 소요시간

🎯 4 (복잡) | ⏱️ ~60분

관련된 가능성 있는 PR

🐰 리뷰 모달이 자동으로 팡 열리고,
탭은 잠금장치를 자랑해요.
일주일 창에서 스터디 완료 축하!
memberId는 흐르고, Props는 정확해지고,
오늘도 검토는 우리 함께 합니다. 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 9.09% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title '후기 모달 안 뜨는 부분 수정' directly describes the main objective of the changeset: fixing the issue where the review modal does not appear, which is confirmed by commit messages and file changes.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/study-review

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.

@HA-SEUNG-JEONG HA-SEUNG-JEONG merged commit 9b59712 into main Mar 25, 2026
7 of 8 checks passed
@HA-SEUNG-JEONG HA-SEUNG-JEONG deleted the hotfix/study-review branch March 25, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant