Skip to content

🐛 대학 상세페이지 진입 실패 시 전역 에러 토스트가 중복 표시되던 문제 수정 - #628

Merged
manNomi merged 1 commit into
mainfrom
fix/university-detail-csr-fallback-toast
Aug 4, 2026
Merged

🐛 대학 상세페이지 진입 실패 시 전역 에러 토스트가 중복 표시되던 문제 수정#628
manNomi merged 1 commit into
mainfrom
fix/university-detail-csr-fallback-toast

Conversation

@manNomi

Copy link
Copy Markdown
Contributor

문제

대학 상세페이지(/university/:homeUniversity/:id) 진입 시 데이터 조회에 실패하면, 화면에는 "대학 정보를 준비중입니다" 같은 안내가 정상적으로 뜨는데 동시에 전역 에러 토스트("오류가 발생했습니다. 다시 시도해주세요.")까지 함께 표시됩니다.

원인

#623에서 추가된 CSR 폴백(UniversityDetailCsrFallback)이 useGetUniversityDetail 훅(react-query)을 쓰는데, apps/university-web의 전역 QueryCache.onError(queryClient.ts)는 meta.skipGlobalErrorToast가 없는 모든 쿼리 실패에 토스트를 띄웁니다. 이 훅에는 그 메타가 없어서, 폴백 UI가 실패를 이미 부드럽게 안내하고 있는데도 토스트가 중복으로 얹혔습니다.

useGetUniversityDetail은 저장소 전체에서 이 CSR 폴백 한 곳에서만 쓰입니다 — 정확히 "대학 상세페이지 진입 실패" 시나리오입니다.

수정

useGetUniversityDetailuseQuery 옵션에 meta: SKIP_GLOBAL_ERROR_TOAST_META를 추가했습니다. apps/university-web에 이미 있는 기존 패턴(postUploadProfileImage.ts 등)을 그대로 따랐습니다.

화면에 보이는 안내(UniversityDetailPreparingFallback)는 그대로 유지되고, 중복 토스트만 사라집니다.

검증

  • pnpm --filter @solid-connect/university-web run lint:check — 321 files 통과
  • pnpm --filter @solid-connect/university-web run typecheck:ci — 통과
  • metaquery.metashouldSkipGlobalErrorToast 로 이어지는 경로를 코드로 재확인

범위 밖 (참고)

같은 페이지의 목록 CSR 폴백(UniversityListCsrFallback)도 인라인 useQuery로 동일한 전역 토스트 문제를 가질 수 있습니다. 다만 이번 요청 범위(상세페이지)에 맞춰 이 PR에는 포함하지 않았습니다. 필요하시면 별도 PR로 처리하겠습니다.

🤖 Generated with Claude Code

@manNomi
manNomi requested a review from wibaek as a code ownerAugust 4, 2026 03:44
@vercel

vercelBot commented Aug 4, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
solid-connect-university-webReadyReadyPreviewAug 4, 2026 3:45am
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
solid-connect-web-adminSkippedSkippedAug 4, 2026 3:45am
solid-connection-webSkippedSkippedAug 4, 2026 3:45am

@github-actionsgithub-actionsBot added the university Changes in apps/university-web label Aug 4, 2026
@coderabbitai

coderabbitaiBot commented Aug 4, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a892a61b-b28d-45b2-a13e-352426a0e9e7

📥 Commits

Reviewing files that changed from the base of the PR and between 08cbe59 and 5a79613.

📒 Files selected for processing (1)
  • apps/university-web/src/apis/universities/getUniversityDetail.ts

Walkthrough

1. **대학 상세 조회 오류 처리 변경** `useGetUniversityDetail`에 `SKIP_GLOBAL_ERROR_TOAST_META`를 추가했습니다. CSR 폴백 사용과 중복 오류 토스트 방지 동작을 설명하는 주석도 추가했습니다.

Estimated code review effort: 2 (Simple) | ~5 minutes

Suggested reviewers:wibaek, khwww, enunsnv

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Title check✅ Passed대학 상세페이지 실패 시 중복 전역 에러 토스트를 제거하는 핵심 변경을 명확히 설명합니다.
Description check✅ Passed문제, 원인, 수정 내용, 검증 결과, 범위 밖 항목을 포함해 변경 목적과 영향을 명확히 설명합니다.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.
✨ 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/university-detail-csr-fallback-toast

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.

@manNomi
manNomi merged commit 62477e3 into mainAug 4, 2026
13 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

universityChanges in apps/university-web

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@manNomi