Repository navigation
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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. Comment |
github-actions
Bot
requested review from
leegain1,
sangrae2325,
tnals0924 and
xeoxxn
October 8, 2026 07:41
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
🎯 해결하려는 문제가 무엇인가요?
홈·행사·공지·열린피드백·빌릴게 헤더에 알림 아이콘이 있는데 눌러도 아무 일도 일어나지 않습니다. Figma 알림 섹션(node-id=3595-106243)의 알림 목록, 알림 없음, 알림 설정 화면을 구현합니다.
❓ 왜 해결해야 하나요?
알림 진입점이 이미 화면에 노출되어 있어서, 사용자가 대여·행사·공지·피드백·사물함 소식을 한곳에서 확인하고 받을 알림을 고를 수 있어야 합니다.
⭐ 어떻게 해결했나요?
/notifications알림 목록과/notifications/settings알림 설정 라우트를 추가했습니다. 둘 다 Bottom Nav 없이 흰 배경입니다.Atomic/Blue/99배경이고, 항목을 누르면 읽음 처리됩니다. 알림이 0개면 알림 없음 화면을 보여주고, 데이터 로딩 중에는 스켈레톤을 보여줍니다.Switch size="small")입니다. "전체 알림"은 따로 값을 두지 않고 카테고리가 모두 켜졌는지로 파생하며, 누르면 4개를 한꺼번에 맞춥니다./notifications로 연결했습니다(viewTransition슬라이드 전환).entities/notificationsAPI +mockupApi+use()/Suspense)를 따릅니다.알림 ↔ notifications를 추가했고,wds-component-usage.md에 알림 화면 WDS 매핑을 기록했습니다. 안 읽음 배경은index.css에--color-notification-unread토큰으로 추가했습니다.커밋 구성: 문서 → 알림 목데이터·API → 알림 목록·설정 화면 → 헤더 알림 아이콘 연결
🧩 이 PR의 한계 & 트레이드오프
fetchNotifications()가mockupApi로 Figma의 6건을 돌려줍니다. API 연동 시 이 함수 안쪽만 교체하면 됩니다. 시간 표시(1시간 전)는 이미 포맷된 문자열로 두었고, 실 API가 시각을 주면 화면에서 상대 시간으로 바꿔야 합니다.notifications는 제가 정했습니다. 백엔드 엔드포인트 네이밍과 맞춰야 하고, 다르면 라우트·폴더 이름을 바꿔야 합니다.알림 - …24px 배지를 SVG로 받았습니다.download_assets내보내기에 컴포넌트 세트 배경이 같이 섞여 있어서, 배지 그룹만 꺼내 24×24 SVG로 다시 감쌌습니다. 모양·색은 Figma 4배 내보내기와 나란히 놓고 비교해 같음을 확인했습니다.NotificationsEmptyState)은 행사·검색 empty와 뼈대가 거의 같지만 공용 컴포넌트로 합치지는 않았습니다.⛓️ 기존 기능에 미치는 영향
onClick만 추가했습니다.index.css에 토큰 1개를 추가했고, 기존 화면의 스타일은 바뀌지 않습니다./notifications,/notifications/settings라우트 2개가 추가됩니다.🔀 Edge Case & 실패 시나리오
replace).📋 검토한 대안과 선택 이유
List/Card사용: 카테고리 배지 + 라벨 + 시간 줄과 들여쓴 메시지 줄 구조가 달라 Stream 로컬 컴포넌트(NotificationsItem)로 만들었습니다. 텍스트는Typography로 채웠습니다.IconMegaphone·IconTicket사용: 배경 배지(#F7F7F8, radius 6)와 색이 종류마다 고정이라 배지 전체를 로컬 에셋으로 썼습니다.💬 리뷰 포인트
[r]용어notifications가 백엔드 엔드포인트 네이밍과 맞는지 확인해 주세요.[c]"전체 알림"과 카테고리 스위치의 연동 규칙(전체를 끄면 카테고리도 모두 끄는지 등)이 의도와 맞는지 봐주세요.[c]알림 항목 탭 시 읽음 처리만 하고 이동하지 않는 동작이 맞는지 확인해 주세요.[a]이 환경에서는 Pretendard 폰트가 로드되지 않아 대체 폰트 기준으로 줄바꿈과 폭을 확인했습니다. 실기기에서 줄바꿈이 Figma와 같은지 한 번 봐주세요.