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

kuring-73 멀티 모듈 Feature 에서 화면 Navigate 하는 레퍼런스 추가 #67

Closed
wants to merge 4 commits into from

Conversation

yeon-kyu
Copy link
Member

@yeon-kyu yeon-kyu commented Aug 30, 2023

https://kuring.atlassian.net/browse/KURING-73

이 PR은 merge 하지 않을 예정입니다

변경 내용

  • Feature A 에서 Feature B 를 로 직접적인 의존성을 갖지 않고, App Layer의 Navigator로 접근하는 인터페이스를 주입받아 Feature간 이동을 하도록 변경합니다.
  • 현재는 App Layer의 Actvity에서 TestFeatureActivity로 이동하지만, App Layer 가 아닌 임의의 Feature 레이어에서도 Common 모듈에 대한 의존성만 있어도 다른 Feature로 navigate 할 수 있습니다

navigator를 추가하는 f57e737 와 이를 사용하는 b0f6898 커밋 위주로 보시면 빠르게 리뷰가 가능할것 같습니다

Copy link
Member

@mwy3055 mwy3055 left a comment

Choose a reason for hiding this comment

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

이해했습니다. 일단 지금은 이렇게 하고, compose migration이 끝난 후에 다시 논의하면 되겠네요.

앞으로의 작업 순서는 다음과 같습니다.

  1. 화면 Navigation 인터페이스, 구현체 정의하기 (KuringNavigator?)
  2. navigation 인터페이스만을 통해서만 화면이 전환되도록 수정하기
  3. ReEngagementNotificationWorker이 lambda를 매개변수로 받을 수 있도록 수정하기 (이 글 참고)
  4. 모듈화 계속

@yeon-kyu yeon-kyu closed this Sep 1, 2023
@github-actions github-actions bot deleted the task/kuring-73_YK branch April 8, 2024 12:18
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.

2 participants