[캠퍼스/Hotfix] 교내 시설물 정보 웹뷰 페이지에서의 아이콘 미노출 버그 수정

[ff1451]

• Close [캠퍼스] 교내 시설물 정보 웹뷰 페이지 아이콘 미노출 수정 #1215

What is this PR? 🔍

• 기능 : 버그 수정
• issue : [캠퍼스] 교내 시설물 정보 웹뷰 페이지 아이콘 미노출 수정 #1215

Changes 📝

겨울 방학 동안 복지관 편의점 공사로 인해 분리되어 내려오던 API 응답값이 공사 종료로 통합되어 이에 맞춰 수정합니다.

ScreenShot 📷

Precaution

develop 브랜치에 별도 작업이 남아있는 것이 없어 develop에 머지 후 바로 main에 머지하겠습니다.

✔️ Please check if the PR fulfills these requirements

• It's submitted to the correct branch, not the develop branch unconditionally?
• If on a hotfix branch, ensure it targets main?
• There are no warning message when you run yarn lint

Summary by CodeRabbit

릴리스 노트

• 버그 수정
  • 캠퍼스 편의점 아이콘 표시 관련 시설 정보 매핑을 개선했습니다.

[coderabbitai]

Walkthrough

백엔드 응답값 변경에 대응하기 위해 CampusInfo 컴포넌트의 편의점 아이콘 매핑을 수정했습니다. 기존 두 개의 별도 키를 하나의 통합된 키로 변경하여 아이콘 노출 문제를 해결했습니다.

Changes

Cohort / File(s)	Summary
아이콘 매핑 업데이트 src/components/CampusInfo/index.tsx	SHOP_ICON 매핑에서 '복지관 편의점', '참빛관 편의점' 키를 제거하고 '복지관 참빛관 편의점' 단일 키로 통합. 동일하게 <Cafe /> 아이콘을 가리킵니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경 사항과 정확하게 일치합니다. 아이콘 미노출 버그를 수정하는 핫픽스이며, 캠퍼스 정보 웹뷰 페이지 관련 변경 내용을 명확하게 설명합니다.
Linked Issues check	✅ Passed	연결된 이슈 #1215의 요구사항을 완벽하게 충족합니다. API 응답 통합에 맞춰 편의점 카드 아이콘 매핑을 수정하여 아이콘 미노출 버그를 해결했습니다.
Out of Scope Changes check	✅ Passed	모든 변경 사항이 이슈 #1215의 범위 내에 있습니다. SHOP_ICON 매핑 업데이트만 수행되었으며, 다른 로직이나 렌더링 흐름의 불필요한 변경은 없습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ 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/#1215/webview-campus-info-icon

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[kongwoojin]

대 준 영
고생하셨습니다

[coderabbitai]

🧹 Nitpick comments (1)

src/components/CampusInfo/index.tsx (1)

25-25: 통합 키만 남기면 응답 혼재 구간에서 아이콘 공백이 재발할 수 있습니다.

Line 25 수정 자체는 맞지만, 기존 키(복지관 편의점, 참빛관 편의점)를 제거하면 배포 전후/캐시 데이터 혼재 시 다시 undefined 아이콘이 렌더링될 수 있습니다. 전환 기간에는 별칭 키를 함께 유지하는 쪽이 더 안전합니다.

호환성 보강 제안

 const SHOP_ICON = {
   서점: <Book />,
   대즐: <Cafe />,
   미용실: <Cut />,
   세탁소: <Laundry />,
   우체국: <PostOffice />,
+  '복지관 편의점': <Cafe />,
+  '참빛관 편의점': <Cafe />,
   '복지관 참빛관 편의점': <Cafe />,
   복사실: <Print />,
   학생식당: <Flatware />,

As per coding guidelines src/**: "가독성·안정성(Null/에러 처리)·테스트/유지보수 용이성·브라우저/접근성 이슈 등을 검토해주세요."

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/components/CampusInfo/index.tsx` at line 25, The change replaced multiple
cafe keys with a single unified key but removing the old keys can cause
undefined icons during cache/rollout overlap; in
src/components/CampusInfo/index.tsx keep the legacy keys ('복지관 편의점', '참빛관 편의점')
as aliases mapping to the same component (e.g., ensure entries like '복지관 편의점':
<Cafe /> and '참빛관 편의점': <Cafe /> remain alongside '복지관 참빛관 편의점': <Cafe />), add
a short comment marking them as deprecated aliases for rollout safety, and only
remove them in a future cleanup once old clients/caches are guaranteed cleared
(update any related lookup logic in the CampusInfo component to prefer the
unified key but tolerate aliases).

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/components/CampusInfo/index.tsx`:
- Line 25: The change replaced multiple cafe keys with a single unified key but
removing the old keys can cause undefined icons during cache/rollout overlap; in
src/components/CampusInfo/index.tsx keep the legacy keys ('복지관 편의점', '참빛관 편의점')
as aliases mapping to the same component (e.g., ensure entries like '복지관 편의점':
<Cafe /> and '참빛관 편의점': <Cafe /> remain alongside '복지관 참빛관 편의점': <Cafe />), add
a short comment marking them as deprecated aliases for rollout safety, and only
remove them in a future cleanup once old clients/caches are guaranteed cleared
(update any related lookup logic in the CampusInfo component to prefer the
unified key but tolerate aliases).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: cac166d9-d088-446d-b5e1-0e00e8cce5fe

📥 Commits

Reviewing files that changed from the base of the PR and between 04d4edb and 73c7284.

📒 Files selected for processing (1)

• src/components/CampusInfo/index.tsx