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

[KMM Support] Support using Koin for DI instead of Hilt (Dagger) #514

Closed
Tracked by #506
luongvo opened this issue Aug 31, 2023 · 0 comments · Fixed by #510
Closed
Tracked by #506

[KMM Support] Support using Koin for DI instead of Hilt (Dagger) #514

luongvo opened this issue Aug 31, 2023 · 0 comments · Fixed by #510
Assignees
@luongvo
Labels
type : feature
Milestone
3.25.0

Comments

@luongvo
Copy link
Member

luongvo commented Aug 31, 2023 •
edited
Loading

Why

As presented via this presentation, we need to support using Koin for DI instead of Hilt (Dagger) to support the KMM Templates.

  • Hilt does not support KMM yet (Feature request: Support KSP google/dagger#2349).
  • Koin is the recommended DI for KMM.
  • All project component’s structure will remain. Only DI configuration changes.
  • DI with Koin will kept in a separate branch as a second DI option to switch from the script.

Who Benefits?

Developers
@luongvo luongvo mentioned this issue Aug 31, 2023
Closed
@luongvo luongvo self-assigned this Aug 31, 2023
@luongvo luongvo added this to the 3.24.0 milestone Aug 31, 2023
@luongvo luongvo mentioned this issue Aug 31, 2023
Merged
@luongvo luongvo linked a pull request Aug 31, 2023 that will close this issue
[#514] [KMM Support] Migrate to Hilt to Koin #510
Merged
@luongvo luongvo changed the title Support using Koin for DI instead of Hilt (Dagger) [KMM Support] Support using Koin for DI instead of Hilt (Dagger) Aug 31, 2023
luongvo added a commit that referenced this issue Aug 31, 2023
@luongvo
[#514] Migrate Hilt to Koin
4b807e2
luongvo added a commit that referenced this issue Aug 31, 2023
@luongvo
[#514] Update tests for Koin
38b8afd
luongvo added a commit that referenced this issue Aug 31, 2023
@luongvo
[#514] Use factory scope for usecases, and repositories
44b1269
luongvo added a commit that referenced this issue Aug 31, 2023
@luongvo
[#514] Move all network dependencies to data layer: OkHttp + Chucker
168991c
luongvo added a commit that referenced this issue Aug 31, 2023
@luongvo
[#514] Remove redundant kapt usage
f7a0215
luongvo added a commit that referenced this issue Aug 31, 2023
@luongvo
[#514] Move LocalModule down to the data module
65615c1
luongvo added a commit that referenced this issue Aug 31, 2023
@luongvo
[#514] Resolve packages after rebasing
2d0a4a0
luongvo added a commit that referenced this issue Oct 2, 2023
@luongvo
[#514] Migrate Hilt to Koin
67825a0
luongvo added a commit that referenced this issue Oct 2, 2023
@luongvo
[#514] Update tests for Koin
440fabe
luongvo added a commit that referenced this issue Oct 2, 2023
@luongvo
[#514] Use factory scope for usecases, and repositories
1b15337
luongvo added a commit that referenced this issue Oct 2, 2023
@luongvo
[#514] Move all network dependencies to data layer: OkHttp + Chucker
e04a880
luongvo added a commit that referenced this issue Oct 2, 2023
@luongvo
[#514] Remove redundant kapt usage
0e7859d
luongvo added a commit that referenced this issue Oct 2, 2023
@luongvo
[#514] Move LocalModule down to the data module
16e5724
luongvo added a commit that referenced this issue Oct 2, 2023
@luongvo
[#514] Resolve packages after rebasing
9b79617
@ryan-conway ryan-conway modified the milestones: 3.24.0, 3.25.0 Oct 5, 2023
luongvo added a commit that referenced this issue Oct 6, 2023
@luongvo
[#514] Upgrade Koin version
b5f1fd6
ryan-conway added a commit that referenced this issue Oct 12, 2023
@ryan-conway
Merge pull request #510 from nimblehq/feature/replace-hilt-with-koin
7f0a2f6 
[#514] [KMM Support] Migrate to Hilt to Koin
@ryan-conway ryan-conway mentioned this issue Oct 31, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@luongvo luongvo

Labels
type : feature
Projects
None yet
Milestone
3.25.0
Development

Successfully merging a pull request may close this issue.

[#514] [KMM Support] Migrate to Hilt to Koin nimblehq/android-templates
2 participants
@luongvo @ryan-conway