Skip to content

Conversation

@taehwandev
Copy link
Member

Issue

Overview (Required)

  • 라우팅 추가, 세션/컨트리뷰트 이동 일부 적용
  • Route 모듈 : 라우팅 공통화를 위한 모듈
  • Route API 모듈 : feature에서 활용할 route 정의용
  • feature API 모듈 : Route api를 활용해 Route를 정의하며, 이를 기반으로 navigate를 정의

기존 방식

  • Main activity에서부터 route를 처리하기위한 Higher-order 함수를 전달해야하며,
  • navigator 모듈에서 새로운 화면이 추가될때마다 route 경로를 추가해야 함.
  • 새로운 feature 추가 시 매번 navigator를 수정해야하며, 라우팅을 추가해야 함.

변경 방식

  • feature의 api와 Navigator 만 알고있다면 move(Route), moveBack()를 통해 빠른 개발 가능
    • ViewModel의 의존성이 생김 : View > ViewModel을 통해 Navigator를 처리해야 함

@taehwandev taehwandev requested a review from workspace May 14, 2025 12:23
@taehwandev taehwandev self-assigned this May 14, 2025
@taehwandev taehwandev added enhancement New feature or request 2025 labels May 14, 2025
@github-actions
Copy link

github-actions bot commented May 14, 2025

Test Results

29 tests   29 ✅  20s ⏱️
17 suites   0 💤
17 files     0 ❌

Results for commit 0c48d85.

♻️ This comment has been updated with latest results.


import com.droidknights.app.core.router.api.model.Route

interface Navigation {
Copy link
Contributor

Choose a reason for hiding this comment

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

마지막! 요거는 원래 이름인 Navigator가 좀 더 나은 것 같은데 어떻게 생각하세용?

@taehwandev taehwandev merged commit 40e3672 into droidknights:2025/app May 25, 2025
3 checks passed
@taehwandev taehwandev mentioned this pull request May 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2025 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants