Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TNT-208] 트레이너/트레이니 메인 탭 작성 #55

Merged
merged 8 commits into from
Feb 5, 2025

Conversation

FpRaArNkK
Copy link
Contributor

📌 What is the PR?

  • 앱 전체 코디네이트를 위한 트레이너/트레이니 메인 탭을 작성했습니다.

🪄 Changes

  • 신규 디자인 시스템에 맞춰 탭 디자인을 반영했습니다.
  • 각 탭을 포함한 트레이너/트레이니 탭바 뷰를 작성했습니다

🌐 Common Changes

  • 기존의 OnboardingFeature/View의 네이밍을 LoginFeature/View로 수정했습니다.
    • 기존의 네비게이터 역할을 분리하고, SNS 로그인의 역할만 수행하도록 했습니다.
    • 또한 추후 온보딩 플로우 관련 화면들을 관리하는 코디네이터의 이름인 OnboardingFlowFeature의 네이밍을 위함입니다.
    • 기존에 OnboardingFeature에서 담당했던 네비게이션 로직은 OnboardingFlowFeature에서 담당할 예정입니다.
  • TabInfo를 TabBar 폴더로 이동하여 같이 관리되도록 했습니다.

🔥 PR Point

  1. 탭뷰를 담당하는 Feature의 State는 Struct가 아닌 enum 타입으로 관리됩니다.
    각 case는 표시될 화면의 State를 가지며, Reduce에서 ifCaseLet을 통해 리듀서를 주입받습니다.

  2. Action의 하위 enum으로 분리한 SubFeatureAction은 하위 화면에서 발생한 Action을 지칭하며, delegate와 동일한 역할을 수행합니다.

  3. setNavigating Action은 RoutingScreen enum을 연관값으로 갖습니다.
    이는 본 Reducer를 포함한 상위 Reducer에서 setNavigating(let screen)을 감시하고, 각 화면 분기처리를 진행시키기 위함입니다.
    하위 화면에서는 다음 화면을 직접 참조하여 넘기는 것이 아니라, RoutingScreen enum-case를 통해 목적지만 명시합니다. 상위 화면에서는 해당 case를 받아 적절한 화면 처리를 진행합니다.
    -> 다음 PR에서 확인하실 수 있습니다.

📸 Screenshot

기능 스크린샷
GIF
GIF

🙆🏻 To Reviewers

  • 가능한 빨리 확인 부탁드려요!

💭 Related Issues

@FpRaArNkK FpRaArNkK added the ✨Feat 새로운 기능 구현 (새로운 로직 추가, UI 구현 등) label Feb 5, 2025
@FpRaArNkK FpRaArNkK requested a review from syss220211 February 5, 2025 12:33
@FpRaArNkK FpRaArNkK self-assigned this Feb 5, 2025
Copy link
Member

@syss220211 syss220211 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!

@FpRaArNkK FpRaArNkK merged commit af7ea5e into develop Feb 5, 2025
@FpRaArNkK FpRaArNkK requested a review from stealmh February 6, 2025 11:29
@FpRaArNkK FpRaArNkK deleted the TNT-208-mainTab branch February 12, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feat 새로운 기능 구현 (새로운 로직 추가, UI 구현 등)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants