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.

Sign up for GitHub

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

Jump to bottom

🔀 :: Moordinator 2.0.0 버전 대응 및 Interface 모듈 제거 #21

Merged
merged 6 commits into from
May 10, 2023
Merged

🔀 :: Moordinator 2.0.0 버전 대응 및 Interface 모듈 제거 #21

merged 6 commits into from
May 10, 2023

Conversation

baekteun
Copy link
Member

💡 개요

  • 기존의 Moordinator 1.x.x 버전은 Router를 외부에서 주입을 해야만 했습니다, 단 이러면 Navigator 패턴도 아니면서 Coordinator 패턴도 아닌 어중간한 패턴이 되어버리고, 모든 Interface모듈 Moordinator를 의존해야했습니다.
  • Navigator와 Coordinator중 Coordinator 방식을 선정하였고, 이를 사용할때 Interface모듈은 불필요한 모듈이기에 제거합니다.

📃 작업내용

  • 모든 Feature모듈의 Interface모듈을 제거
  • Coordinator를 따르는 방식으로 Moordinator를 사용

🔀 변경사항

  • Base에서 일부 클래스들을 조금 더 protocol 지향적으로 변경
  • Interface모듈 제거
  • Router를 주입이 아닌 로직 객체에서 처리하도록 변경

@baekteun baekteun added 🔨 Refactor 코드 리팩토링 0️⃣Priority: Critical 우선순위 긴급 labels May 10, 2023
@baekteun baekteun self-assigned this May 10, 2023
@baekteun baekteun linked an issue May 10, 2023 that may be closed by this pull request
Moordinator 버전 2.0 업그레이드 #18
Closed
@baekteun baekteun merged commit a3b5ab4 into master May 10, 2023
@kimsh153 kimsh153 deleted the 18-moordinator-2-0-upgrade branch May 10, 2023 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kimsh153 kimsh153 kimsh153 approved these changes

@minaje0917 minaje0917 minaje0917 approved these changes

Assignees

@baekteun baekteun

Labels
0️⃣Priority: Critical 우선순위 긴급 🔨 Refactor 코드 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Moordinator 버전 2.0 업그레이드
3 participants
@baekteun @minaje0917 @kimsh153