Skip to content

feat: 멘토 페이지 로딩 스켈레톤 적용#548

Merged
manNomi merged 1 commit into
mainfrom
feat/mentor-ssr-hydration
Jun 3, 2026
Merged

feat: 멘토 페이지 로딩 스켈레톤 적용#548
manNomi merged 1 commit into
mainfrom
feat/mentor-ssr-hydration

Conversation

@manNomi
Copy link
Copy Markdown
Contributor

@manNomi manNomi commented Jun 3, 2026

관련 이슈

  • 없음

작업 내용

  • 멘토 페이지의 인증 확인 대기, 멘티/멘토 탭, 멘토 찾기 목록, 내 멘토 프로필, 신청 목록에 화면 구조를 유지하는 스켈레톤을 추가했습니다.
  • /mentor route-level loading.tsx를 추가해 라우트 전환 중에도 멘토 페이지 형태의 로딩 화면이 보이도록 했습니다.
  • 멘토 찾기 목록이 로딩 중일 때 빈 상태 문구가 먼저 보이지 않도록 isPending 기준 분기를 추가했습니다.
  • 멘토 찾기 탭 프리페치 콜백을 useCallback으로 안정화해 effect가 불필요하게 다시 예약되지 않도록 했습니다.
  • 목록 마지막 아이템 감지에서 indexOf 대신 map의 index를 사용하도록 정리했습니다.

특이 사항

  • 멘토 관련 API는 인증 없이 호출하면 401을 반환하고, isApplied처럼 사용자별 상태를 포함할 수 있어 커뮤니티 페이지처럼 서버 prefetch hydration을 직접 적용하지 않았습니다.
  • true authenticated SSR hydration은 refresh token 재발급, access token 전달, Set-Cookie 전파 정책까지 별도 설계가 필요해서 이번 PR 범위에서는 제외했습니다.
  • 재사용을 위해 로컬 Codex 스킬 page-hydration-skeleton~/.codex/skills/page-hydration-skeleton에 추가했습니다. 이 스킬 파일은 개인 로컬 Codex 설정이라 저장소 PR에는 포함되지 않습니다.
  • 로컬 Node 버전이 프로젝트 권장값(22.x)과 달라 Unsupported engine 경고가 출력되지만 검증은 통과했습니다.

리뷰 요구사항 (선택)

  • 인증 API를 서버에서 무리하게 prefetch하지 않고 skeleton 중심으로 처리한 범위 판단이 적절한지 봐주세요.
  • 각 탭/목록의 pending 상태에서 empty state가 먼저 보이지 않는지 확인 부탁드립니다.

검증

  • pnpm --filter @solid-connect/web run lint:check
  • pnpm --filter @solid-connect/web run typecheck:ci
  • pnpm --filter @solid-connect/web run build
  • push hook의 ci:checkbuild 통과
  • 브라우저에서 비로그인 상태 /mentor 접근 시 /login 정상 이동 및 콘솔 에러 없음 확인
  • page-hydration-skeleton 스킬 quick_validate.py 통과

@manNomi manNomi requested review from enunsnv and wibaek as code owners June 3, 2026 06:13
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Warning

Review limit reached

@manNomi, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 54 minutes and 7 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 4a3ed409-d049-4bdd-a93d-781e7df4bc25

📥 Commits

Reviewing files that changed from the base of the PR and between d0779ed and 3ab4268.

📒 Files selected for processing (9)
  • apps/web/src/apis/mentor/getMentorList.ts
  • apps/web/src/app/mentor/_ui/MentorClient/_ui/MenteePageTabs/index.tsx
  • apps/web/src/app/mentor/_ui/MentorClient/_ui/MentorFindSection/index.tsx
  • apps/web/src/app/mentor/_ui/MentorClient/_ui/MentorPage/_ui/ApplicantListSection/index.tsx
  • apps/web/src/app/mentor/_ui/MentorClient/_ui/MentorPage/_ui/MyMentorSection/index.tsx
  • apps/web/src/app/mentor/_ui/MentorClient/_ui/MentorPage/index.tsx
  • apps/web/src/app/mentor/_ui/MentorClient/index.tsx
  • apps/web/src/app/mentor/_ui/MentorPageSkeleton.tsx
  • apps/web/src/app/mentor/loading.tsx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/mentor-ssr-hydration

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.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 3, 2026

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

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

@github-actions github-actions Bot added the web label Jun 3, 2026
@manNomi manNomi merged commit 74e9152 into main Jun 3, 2026
10 of 11 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