Skip to content

fix: 멘토 채팅 화면 버그 수정#562

Merged
manNomi merged 1 commit into
mainfrom
fix/issue-505-mentor-chat
Jun 21, 2026
Merged

fix: 멘토 채팅 화면 버그 수정#562
manNomi merged 1 commit into
mainfrom
fix/issue-505-mentor-chat

Conversation

@manNomi

@manNomi manNomi commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

작업 내용

  • 멘토 채팅 메시지의 상대방 프로필에 실제 파트너 프로필 이미지를 전달하도록 수정했습니다.
  • 프로필 이미지가 없을 때는 기존 기본 프로필 이미지 fallback이 노출되도록 유지했습니다.
  • 채팅방 최초 진입 시 메시지 목록이 하단으로 안정적으로 이동하도록 초기 스크롤 보정 로직을 강화했습니다.

특이 사항

  • env 파일은 기존 작업본에서 새 클론으로 복사했으며 git 추적 대상에는 포함하지 않았습니다.
  • 로컬 Node 버전이 repo 권장값인 22.x가 아닌 v23.10.0이라 엔진 경고가 출력되지만, 검증은 통과했습니다.

검증:

  • pnpm --filter @solid-connect/web run lint:check
  • pnpm --filter @solid-connect/web run typecheck
  • commit/push hook의 @solid-connect/web ci:check
  • push hook의 @solid-connect/web build
  • curl -I http://localhost:3000/mentor/chat/1 200 응답 확인

리뷰 요구사항 (선택)

  • 실제 API 데이터에서 채팅 상대 프로필 이미지와 기본 이미지 fallback이 디자인 기대와 맞는지 확인 부탁드립니다.
  • 이미지/메시지 렌더링 후 최초 하단 스크롤 위치가 안정적인지 확인 부탁드립니다.

@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
solid-connect-university-web Error Error Jun 19, 2026 7:46am
solid-connection-web Ready Ready Preview, Comment Jun 19, 2026 7:46am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jun 19, 2026 7:46am

@coderabbitai

coderabbitai Bot commented Jun 19, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b5b06ed2-2053-4257-8a1a-f556044292ac

📥 Commits

Reviewing files that changed from the base of the PR and between c00c55a and b720054.

📒 Files selected for processing (3)
  • apps/web/src/app/mentor/chat/[chatId]/_ui/ChatContent/_hooks/useChatListHandler.ts
  • apps/web/src/app/mentor/chat/[chatId]/_ui/ChatContent/_ui/ChatMessageBox/index.tsx
  • apps/web/src/app/mentor/chat/[chatId]/_ui/ChatContent/index.tsx

Walkthrough

이번 PR에는 두 가지 독립적인 변경 사항이 포함되어 있습니다.

  1. 초기 자동 스크롤 예약 전략 개선

    • INITIAL_SCROLL_SETTLE_DELAYS_MS 상수가 추가됩니다.
    • 기존 단일 requestAnimationFrame 방식이, 2회 연속 RAF + INITIAL_SCROLL_SETTLE_DELAYS_MS 배열 기반 다중 setTimeout 방식으로 교체됩니다.
    • 스크롤 완료 플래그 기록 타이밍이 "첫 RAF 실행 시"에서 "예약 직후"로 이동합니다.
    • 초기 스크롤 조건에서 isFetchingNextPage 의존성이 제거됩니다.
    • cleanup 함수에서 모든 RAF와 타이머가 취소됩니다.
  2. 파트너 프로필 이미지 URL 전달 prop 체인 추가

    • ChatMessageBoxPropspartnerProfileUrl?: string | null 필드가 추가됩니다.
    • ChatMessageBox 컴포넌트의 기본값으로 null이 설정됩니다.
    • 파트너 메시지 렌더링 시 ProfileWithBadgeprofileImageUrl={partnerProfileUrl}이 전달됩니다.
    • ChatContent에서 partnerProfileUrl={profileUrl}ChatMessageBox로 전달합니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • wibaek
  • enunsnv
✨ 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 fix/issue-505-mentor-chat

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.

@github-actions github-actions Bot added the web label Jun 19, 2026
@manNomi manNomi marked this pull request as ready for review June 21, 2026 07:05
@manNomi manNomi requested review from enunsnv and wibaek as code owners June 21, 2026 07:05
@manNomi manNomi merged commit cc0ee10 into main Jun 21, 2026
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

멘토 채팅 수정

1 participant