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

feat: BaseNavigator 및 App, Splash, CalendarNavigator 등 코드 작성 #582

Merged
merged 9 commits into from
Jun 27, 2024

Conversation

rlarjsdn3
Copy link
Collaborator

@rlarjsdn3 rlarjsdn3 commented Jun 26, 2024

작업 내용 🧑‍💻

  • BaseNavigator 구현
  • App, Splash, Calendar, Management, CommentNavigator 등 코드 구현
  • SignInViewControllerWrapper 구현

변경 로직 ⚒️

  • AppNavigator는 SceneDelegate에 위치하며 UIWindow에 새로운 UINavigationController를 넣고 첫 화면(Splash)을 띄우는 역할을 합니다.
  • SplashNavigator는 초대 코드, 가족 코드 존재 유무에 따라 Home, SignInViewController로 화면을 전환하는 역할을 합니다.
  • MonthlyCalendarNavigator는 월별 달력 화면에서 일별 달력 화면으로 전환하는 역할을 합니다. 뒤로 가기(BackToHome)도 가능합니다.
  • 이외 Navigator는 작성한 코드와 기능이 동일합니다.

  • 원래 Cell에서 화면 전환을 하려면 귀찮은 ServiceProvider를 통해야 했는데, Navigator를 사용하면 이 같은 귀찮음도 줄어듭니다! (CalendarCellReactor.swift 참조)

주의 사항🔴

  • navigator는 Reactor의 Mutate에서 호출합니다.
  • 화면 전환을 포함하는 모든 ViewController에 대응되는 Navigator를 만들어야 합니다.
  • Navigator 코드 구현만 완료했고, 실제 적용은 아직 하지 않았습니다. AppNavigator는 예시로 보여드리기 위해 적용하였습니다.
  • 코드를 보시면 아시다시피, 우리는 windownavigationController를 넣고, 해당 navigationController로 모든 화면을 전환합니다. 만약, 구현을 하시고 제대로 작동하지 않는다면 SplashViewController에서 window에 새로운 뷰 컨트롤러가 넣어지지 않았는지 확인해주세요.
  • AppNavigator를 제외한 나머지 Navigator에서는 window에 새로운 뷰 컨트롤러를 넣으면 안됩니다!

close #563
close #583

@rlarjsdn3 rlarjsdn3 changed the title feat: BaseNavigator 및 App, Splash, MonthlyCalendarNavigator 예시 코드 작성 feat: BaseNavigator 및 App, Splash, CalendarNavigator 등 코드 작성 Jun 26, 2024
Do-hyun-Kim
Do-hyun-Kim previously approved these changes Jun 27, 2024
Copy link
Collaborator

@Do-hyun-Kim Do-hyun-Kim left a comment

Choose a reason for hiding this comment

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

Navigator 작업하시느라 고생하셨습니다!! 👍 👍

@rlarjsdn3
Copy link
Collaborator Author

@Do-hyun-Kim 감삼돠

@rlarjsdn3 rlarjsdn3 merged commit ecb2e85 into develop Jun 27, 2024
1 of 2 checks passed
@rlarjsdn3 rlarjsdn3 deleted the feat/#563-Navigator구현 branch June 27, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants