Skip to content

refactor: 임시 조치 복구#175

Merged
clxxrlove merged 1 commit intodevelopfrom
refactor/#164
Feb 21, 2026
Merged

refactor: 임시 조치 복구#175
clxxrlove merged 1 commit intodevelopfrom
refactor/#164

Conversation

@clxxrlove
Copy link
Member

🔗 관련 이슈

📙 작업 내역

  • 홈 네비게이션 스타일, 탭바, 알림 센터 복구
  • 커플 페이지는 여전히 비활성 상태로 복구 -> 다음 배포 버전에 미포함 예정이므로

💬 추가 설명 or 리뷰 포인트 (선택)

  • 이거 작업은 해놨는데 푸시를 안한지 모르고 있었네... ;;

@coderabbitai
Copy link

coderabbitai bot commented Feb 21, 2026

📝 Walkthrough

요약

배포를 위해 이전에 비활성화했던 Home 네비게이션, 탭 바, 알림 센터 기능을 복구합니다. 동시에 Settings 탭을 MainTab 레벨에서 제거하고 HomeCoordinator 내 모달로만 접근 가능하도록 아키텍처를 정리합니다.

아키텍처 변경사항

기능 경계 재편성

  • MainTabReducer: Settings 상태 및 .settings 액션 제거

    • 탭 바 노출 범위 축소: .home, .statistics만 공개 (.couple은 내부용)
    • Settings는 이제 HomeCoordinator 내 모달/시트로만 접근 가능
  • MainTabView: FeatureSettings 임포트 제거 및 settings UI 케이스 삭제

상태 및 UI 흐름 단순화

  • SettingsReducer.State: showBackButton 프로퍼티 및 초기화 파라미터 제거

    • SettingsView는 항상 뒤로가기 버튼이 있는 서브타이틀 네비게이션 바 렌더링
  • TXTabItem: .settings 열거형 케이스 제거

    • visibleCases[.home, .statistics]로 변경
    • 관련 아이콘 및 제목 케이스 핸들링 제거

사용자 인터페이스 복구

  • TXNavigationBar (Home 스타일):

    • .alertTapped 버튼 활성화 (알림 센터)
    • .settingTapped 버튼 활성화 (설정)
  • SettingsView:

    • 알림 설정 항목 활성화
    • 상단 콘텐츠 패딩 복구
  • AuthView: Kakao 로그인 버튼 활성화

동작 변화

  • Settings 화면은 Home 탭의 네비게이션 액션으로만 진입 가능 (모달/시트)
  • 메인 탭 바에서 Settings 탭 제거 (다음 릴리스에서 제외 예정, Couple 페이지 포함)
  • Home 화면의 알림 센터 및 설정 버튼 기능 복구

코드 리뷰 포인트

  • HomeCoordinator와 MainTabReducer 간의 기능 경계 명확성
  • SettingsReducer 상태 간소화로 인한 초기화 체인 일관성
  • TXTabItem 변경 사항이 모든 탭 관련 컴포넌트에 반영되었는지 확인 (TXTabBar, TXTabBarContainer 등)
  • Home 네비게이션 액션 흐름과 Settings 모달 닫기 로직의 동작

개요

이전에 배포를 위해 비활성화했던 기능들을 복구합니다: 카카오 로그인 버튼 활성화, 메인 탭에서 설정 화면 제거, 알림 설정 항목 재활성화, 네비게이션 바의 액션 버튼 활성화, 탭 바에서 설정 탭 항목 제거 (설정 리듀서는 내부에서 계속 사용)

변경사항

코호트 / 파일 요약
인증 기능
Projects/Feature/Auth/Sources/View/AuthView.swift
카카오 로그인 버튼의 주석을 해제하여 로그인 버튼 섹션에 포함시킴
메인 탭 기능
Projects/Feature/MainTab/Sources/Reducer/MainTabReducer.swift, Projects/Feature/MainTab/Sources/View/MainTabView.swift
설정 하위 기능을 메인 탭 리듀서에서 제거 (상태, 액션, 자식 리듀서 스코핑 제거) 및 메인 탭 뷰에서 설정 화면 케이스 제거
설정 기능
Projects/Feature/Settings/Interface/Sources/SettingsReducer.swift, Projects/Feature/Settings/Sources/Settings/SettingsView.swift
설정 리듀서에서 showBackButton 프로퍼티 제거, 설정 뷰에서 상단 패딩 재활성화, 네비게이션 바 렌더링 단순화, 알림 설정 항목 재활성화
디자인 시스템 컴포넌트
Projects/Shared/DesignSystem/Sources/Components/Bar/Navigation/TXNavigationBar.swift, Projects/Shared/DesignSystem/Sources/Components/Bar/TabBar/TXTabBar.swift, Projects/Shared/DesignSystem/Sources/Components/Bar/TabBar/TXTabItem.swift
네비게이션 바의 주석 처리된 액션 버튼 활성화 (알림, 설정), 탭 바에서 isSelected 바인딩 제거 및 인라인 비교로 리팩토링, 탭 아이템에서 설정 탭 제거

예상 코드 리뷰 노력

🎯 3 (Moderate) | ⏱️ ~20 minutes

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/#164

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.

@clxxrlove
Copy link
Member Author

@coderabbitai summary

@coderabbitai
Copy link

coderabbitai bot commented Feb 21, 2026

@clxxrlove: PR의 요약을 생성하겠습니다.

✅ Actions performed

Summary regeneration triggered.

@clxxrlove clxxrlove merged commit 85e8d96 into develop Feb 21, 2026
6 checks passed
@clxxrlove clxxrlove deleted the refactor/#164 branch February 21, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant