Skip to content

[fix] 1차 MVP 기준 네비게이션 바 요소 수정#22

Merged
dohy-eon merged 1 commit into
devfrom
fix/ALT-187
May 3, 2026
Merged

[fix] 1차 MVP 기준 네비게이션 바 요소 수정#22
dohy-eon merged 1 commit into
devfrom
fix/ALT-187

Conversation

@dohy-eon
Copy link
Copy Markdown
Member

@dohy-eon dohy-eon commented May 3, 2026

ID

  • ALT-187

변경 내용

  • 하단 네비게이션 바 요소 수정

구현 사항

  • 1차 MVP 기준으로 하단 네비게이션 바에서 소셜/평판 탭을 제거하고, 관련 타입·스토어 상태·라우팅 매핑·스토리북까지 정리했습니다.

구현 시연 (필요 시)

image

Summary by CodeRabbit

릴리스 노트

  • 변경 사항
    • Message 및 Repute 탭이 제거되었습니다.
    • 네비게이션은 이제 Home, My, Search 탭만 표시합니다.
    • 관련 UI 요소 및 라우팅이 정리되었습니다.

@dohy-eon dohy-eon requested review from kim3360 and limtjdghks May 3, 2026 05:06
@dohy-eon dohy-eon self-assigned this May 3, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented May 3, 2026

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

Project Deployment Actions Updated (UTC)
alter-client Ready Ready Preview, Comment May 3, 2026 5:06am

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: eb3be059-e092-4050-8b88-1a3b561f56fa

📥 Commits

Reviewing files that changed from the base of the PR and between 75bdddf and 5672c05.

📒 Files selected for processing (4)
  • src/shared/stores/useDocStore.ts
  • src/shared/types/tab.ts
  • src/shared/ui/common/Docbar.tsx
  • storybook/stories/Docbar.stories.tsx
💤 Files with no reviewable changes (3)
  • storybook/stories/Docbar.stories.tsx
  • src/shared/ui/common/Docbar.tsx
  • src/shared/stores/useDocStore.ts

📝 Walkthrough

Walkthrough

메시지 및 명성 탭을 제거하는 일관된 리팩토링. 타입 정의에서 TabKey 축소, 스토어 상태 단순화, UI 렌더링 제거, 스토리북 예제 정리.

Changes

탭 기능 제거

Layer / File(s) Summary
Data Shape
src/shared/types/tab.ts
TabKey 타입이 'home' | 'my' | 'search'로 축소되고, TAB_TITLE_MAP도 동일하게 업데이트됨.
State Management
src/shared/stores/useDocStore.ts
PATHNAME_TAB_MAP에서 /message, /repute, /manager/social 라우트 제거; createSelectedTabDocStoreState.selectedTab이 세 탭만 포함하도록 단순화.
UI Component
src/shared/ui/common/Docbar.tsx
MessageIcon, ReputeIcon 임포트 제거; 탭 버튼 렌더링에서 message/repute 버튼 삭제; pathByTab 라우팅 맵 축소.
Tests / Stories
storybook/stories/Docbar.stories.tsx
createSelectedTab 헬퍼 업데이트; MessageSelected 스토리 내보내기 제거.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes


시니어 리뷰 관점 — 핵심 이슈

추가 확인 필요:

  1. 미처리된 참조 위험
    useDocStore.setSelectedTabByPathname()이 더 이상 message/repute를 설정할 수 없으므로, 이전 경로(예: /message, /repute)로 직접 접근한 사용자는 어디로 리다이렉트되는가? 404이거나 폴백 로직이 있어야 함.

  2. 타입 안정성 확인
    이 변경이 useDocStore를 사용하는 다른 컴포넌트들에서 selectedTab.message 또는 selectedTab.repute를 참조하는 부분이 없는지 TypeScript 컴파일 결과 확인 필요. PR 범위 외 오류가 없는지 보증.

  3. 라우팅 일관성
    pathByTab에서 제거된 경로들이 애플리케이션 라우터의 다른 부분(예: 404 핸들링, 역방향 네비게이션)에서 처리되고 있는지 확인.

현재 변경은 일관되고 깔끔하지만, 전체 애플리케이션 통합 이후 위 항목들이 검증되어야 합니다.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 핵심 변경사항(1차 MVP 기준 네비게이션 바 요소 수정)을 정확히 반영하고 있습니다.
Description check ✅ Passed PR 설명이 제목, 변경 내용, 구현 사항, 구현 시연을 포함하여 템플릿 구조를 충실히 따르고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ALT-187

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get your free trial and get 200 agent minutes per Slack user (a $50 value).


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
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

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

@dohy-eon dohy-eon merged commit 7a5ebd4 into dev May 3, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant