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

Multiplatform support for navigation-compose #1219

Merged
merged 8 commits into from
Mar 25, 2024

Conversation

MatkovIvan
Copy link
Member

@MatkovIvan MatkovIvan commented Mar 25, 2024

Proposed Changes

  • Commonized navigation-common, navigation-runtime and navigation-compose
  • Currently it's done WITHOUT support of deep-links and (Predictive)BackHandler
  • Migrate mpp's custom navigation to navigation-compose

Testing

Test: run mpp sample. There are no commonized unit tests yet (TBD)
Usage example: https://github.com/MatkovIvan/nav_cupcake

Fixed Issues

Fixes JetBrains/compose-multiplatform#85

** DO NOT SQUASH **

@MatkovIvan MatkovIvan force-pushed the ivan.matkov/navigation-jb-main branch 2 times, most recently from e7b0419 to 113fe26 Compare March 25, 2024 11:08
Change-Id: I665f326038bc773530c2cbec2dd16f5ba98144e6
Change-Id: Ifc350a9bdfeaafe1165eef25d143b261eb17f623

Provide navigator name for multiplatform

Change-Id: If39996aab30aaeb9253ea8ac0412ac9916918cc4

Fixup navigation-common

[navigation-common] Expect `NavDeepLink` in common

Remove annotation from jb navigator
Change-Id: I0a35d82ccdecd78d1034fdf37af741f496676c6f
Change-Id: I0074272aedcd099cb38fd65fc9e61823c4fac3ca
Change-Id: Ice1ecd1e06e1626677809016f8fa5e6ab212931b
[navigation-compose] Expect navigators

[navigation-compose] Provide manual BackStackEntryIdViewModel factory

Workaround for "Symbol is unbound"

[navigation-compose] Fix missing project dependencies
@MatkovIvan MatkovIvan force-pushed the ivan.matkov/navigation-jb-main branch from 113fe26 to c574c9b Compare March 25, 2024 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants